Replace ViewPager2 with Compose-based TaskScreenContainer for Data Collection

[shobhitagarwal1612]

Fixes #3709

This PR overhauls the data collection flow by migrating it to a Compose-based architecture. It removes the legacy ViewPager2 and fragment-based approach (DataCollectionTaskFragment, DataCollectionViewPagerAdapter) in favor of a new TaskScreenContainer. It also refactors event handling to be more deeply integrated with ViewModels.

Key Changes:

• Replace ViewPager2 and task fragments with Compose-based TaskScreenContainer.
• Replace custom TaskMapFragmentContainer with the official Compose AndroidFragment.
• Move map fragment instantiation into individual Compose task screens.
• Simplify event handling in DataCollectionViewModel by removing MutableSharedFlow and using UI effects.
• Remove onButtonClicked parameter from task screens and handle clicks via individual ViewModels.
• Refactor OpenSettings and SetAwaitingPhotoCapture to use DataCollectionViewModel UI effects.
• Fix failing tests and clean up unused imports/parameters.

Steps to verify:

1. Open the app and start a new data collection submission.
2. Verify that you can navigate smoothly between tasks (Next/Previous).
3. Check map-based tasks (Drop Pin, Draw Area, Capture Location) to ensure the map renders correctly and interactions work as expected.
4. Test permission dialogs and photo capture to ensure the new UI effects handle them properly.

@andreia-ferreira PTAL?

[codecov]

Codecov Report

❌ Patch coverage is 64.91228% with 40 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.01%. Comparing base (5c60028) to head (a4d9193).

Files with missing lines	Patch %	Lines
...oid/ui/datacollection/tasks/TaskScreenContainer.kt	52.94%	12 Missing and 4 partials ⚠️
...droid/ui/datacollection/DataCollectionViewModel.kt	35.71%	7 Missing and 2 partials ⚠️
.../android/ui/datacollection/DataCollectionScreen.kt	85.00%	0 Missing and 3 partials ⚠️
...ection/tasks/location/CaptureLocationTaskScreen.kt	50.00%	3 Missing ⚠️
...ndroid/ui/datacollection/DataCollectionFragment.kt	75.00%	1 Missing ⚠️
...ui/datacollection/tasks/AbstractTaskMapFragment.kt	0.00%	0 Missing and 1 partial ⚠️
...oid/ui/datacollection/tasks/date/DateTaskScreen.kt	50.00%	0 Missing and 1 partial ⚠️
...lection/tasks/instruction/InstructionTaskScreen.kt	50.00%	0 Missing and 1 partial ⚠️
...ui/datacollection/tasks/number/NumberTaskScreen.kt	50.00%	0 Missing and 1 partial ⚠️
...d/ui/datacollection/tasks/photo/PhotoTaskScreen.kt	50.00%	0 Missing and 1 partial ⚠️
... and 3 more

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #3735      +/-   ##
============================================
+ Coverage     68.00%   68.01%   +0.01%     
+ Complexity     1613     1596      -17     
============================================
  Files           370      368       -2     
  Lines          9563     9489      -74     
  Branches       1248     1241       -7     
============================================
- Hits           6503     6454      -49     
+ Misses         2381     2366      -15     
+ Partials        679      669      -10     

Files with missing lines	Coverage Δ
...d/ui/datacollection/tasks/AbstractTaskViewModel.kt	93.33% <100.00%> (+0.11%)	⬆️
...oid/ui/datacollection/tasks/DataCollectionEvent.kt	100.00% <100.00%> (ø)
...ion/tasks/location/CaptureLocationTaskViewModel.kt	93.47% <100.00%> (+0.14%)	⬆️
...n/tasks/multiplechoice/MultipleChoiceTaskScreen.kt	52.27% <100.00%> (ø)
...ui/datacollection/tasks/point/DropPinTaskScreen.kt	66.03% <100.00%> (+3.53%)	⬆️
...datacollection/tasks/polygon/DrawAreaTaskScreen.kt	46.73% <100.00%> (+3.06%)	⬆️
...ndroid/ui/datacollection/DataCollectionFragment.kt	92.59% <75.00%> (-3.41%)	⬇️
...ui/datacollection/tasks/AbstractTaskMapFragment.kt	60.27% <0.00%> (ø)
...oid/ui/datacollection/tasks/date/DateTaskScreen.kt	76.56% <50.00%> (-0.37%)	⬇️
...lection/tasks/instruction/InstructionTaskScreen.kt	87.50% <50.00%> (-0.74%)	⬇️
... and 9 more

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[andreia-ferreira]

The only issue I found was when restoring a draft where the current task was conditional. Clicking on any of the previous/next buttons makes the app crash. Tested this behavior on master to confirm it's not an existing issue and couldn't reproduce

Screen_recording_20260513_175154.webm

tested on API 24 and 36 and happens on both

[shobhitagarwal1612]

@andreia-ferreira Thanks for catching the bug! I think it was happening because of using relativeIndex instead of absoluteIndex for rendering tasks. But I was unable to reproduce it locally. Could you please try again with your local setup and check if this resolves the issue?

[andreia-ferreira]

thanks for looking into it, I tested it again but it seems to still happen unfortunately. You can try to join this survey link which I used in the recording below, to see if you can reproduce it too

Screen_recording_20260514_105749.webm