Idrinth / habit-evaluator / 22221205051 / 1

40%

the-one: 40%

push

github

Replace ScrollView with NestedScrollView in all ViewPager2 fragments (#531)

Since lazy-loading was introduced (ec951ff), fragments inside ViewPager2
are no longer pre-loaded. Plain ScrollView doesn't implement the
NestedScrollingParent interface, so scroll events aren't properly
coordinated with ViewPager2's internal RecyclerView. This causes
Maestro's scrollUntilVisible to fail when searching for elements
that require scrolling after a page switch.

The same fix was already applied to fragment_stats.xml in d39b767.
This commit applies it to the remaining 7 fragment layouts:
- fragment_diary_navigation.xml
- fragment_imprint.xml
- fragment_point_development.xml
- fragment_settings.xml
- fragment_record_emotion_entry.xml
- fragment_add_emotion_pair.xml
- fragment_add_habit.xml

https://claude.ai/code/session_01AjZgRtB5NQ8RFP86SFhDD7

Co-authored-by: Claude <noreply@anthropic.com>

60 of 75 branches covered (80.0%)

Branch coverage included in aggregate %.

143 of 153 relevant lines covered (93.46%)

20.66 hits per line