Fix #4072 and Fix Part of #4938: Revised profile chooser UI #5468
Conversation
…reen # Conflicts: # app/src/sharedTest/java/org/oppia/android/app/onboarding/CreateProfileFragmentTest.kt
…le-domain-config # Conflicts: # app/src/main/java/org/oppia/android/app/onboarding/IntroFragment.kt
# Conflicts: # scripts/assets/test_file_exemptions.textproto
|
Unassigning @adhiamboperes since a re-review was requested. @adhiamboperes, please make sure you have addressed all review comments. Thanks! |
Coverage ReportResultsNumber of files assessed: 19 Exempted coverageFiles exempted from coverage
|
APK & AAB differences analysisNote that this is a summarized snapshot. See the CI artifacts for detailed differences. DevExpand to see flavor specificsUniversal APKAPK file size: 19 MiB (old), 19 MiB (new), 40 KiB (Added) APK download size (estimated): 17 MiB (old), 17 MiB (new), 32 KiB (Added) Method count: 260299 (old), 260600 (new), 301 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 6830 (old), 6882 (new), 52 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 18 MiB (old), 18 MiB (new), 39 KiB (Added)
Configuration hdpiAPK file size: 50 KiB (old), 50 KiB (new), 36 bytes (Added) Configuration ldpiAPK file size: 49 KiB (old), 49 KiB (new), 48 bytes (Added) Configuration mdpiAPK file size: 45 KiB (old), 45 KiB (new), 48 bytes (Added) Configuration tvdpiAPK file size: 86 KiB (old), 86 KiB (new), 60 bytes (Added) Configuration xhdpiAPK file size: 57 KiB (old), 57 KiB (new), 36 bytes (Added) Configuration xxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 48 bytes (Added) Configuration xxxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 60 bytes (Added) AlphaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 35 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 26 KiB (Added) Method count: 115822 (old), 116011 (new), 189 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5798 (old), 5841 (new), 43 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 11 MiB (old), 11 MiB (new), 34 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 0 bytes (No change) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 0 bytes (No change) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 0 bytes (No change) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 0 bytes (No change) BetaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 35 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 31 KiB (Added) Method count: 115828 (old), 116017 (new), 189 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5798 (old), 5841 (new), 43 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 11 MiB (new), 34 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 0 bytes (No change) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 0 bytes (No change) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 0 bytes (No change) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 0 bytes (No change) GaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 35 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 31 KiB (Added) Method count: 115828 (old), 116017 (new), 189 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5798 (old), 5841 (new), 43 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 11 MiB (new), 34 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 0 bytes (No change) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 0 bytes (No change) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 0 bytes (No change) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 0 bytes (No change) |
There was a problem hiding this comment.
@BenHenning, PTAL.
I have addressed all pending comments, and update the screenshots in the main PR body. I should be able to upload talkback video tomorrow -- currently facing some physical device issues.
Thanks @adhiamboperes! Took a pass on the new code and left some comments. Just to check, did you address all of my 3 points in the main review message above (quoted from 2 reviews ago)? I did a scan on the images changed in the main PR post, but they don't seem different in any noticeable way and it seems like the original concerns may still be present (minus perhaps Talkback but I don't yet see a video for that to review).
|
|
||
| @Test | ||
| @Config(qualifiers = "land") | ||
| fun testFragment_enableOnboardingV2_rtl_checkRightArrowScrollBehaviourIscorrect() { |
There was a problem hiding this comment.
Are this & the one above actually verifying RTL? Only testFragment_enableOnboardingV2_rtl_checkListIsAlphabetical seems to be enabling RTL explicity.
There was a problem hiding this comment.
I refactored this test to use the Arabic locale.
|
|
||
| launch(ProfileChooserActivity::class.java).use { | ||
| it.onActivity { activity -> | ||
| activity.window.decorView.layoutDirection = ViewCompat.LAYOUT_DIRECTION_RTL |
There was a problem hiding this comment.
Does this actually cause the activity to relayout into RTL?
An alternative that might work is switching the language to Arabic since that should force RTL as well, I think.
There was a problem hiding this comment.
I refactored tests to use the Arabic locale, thanks for the suggestion!
|
|
||
| @Test | ||
| @Config(qualifiers = "land") | ||
| fun testFragment_enableOnboardingV2_rtl_checkLeftArrowScrollBehaviourIscorrect() { |
There was a problem hiding this comment.
| fun testFragment_enableOnboardingV2_rtl_checkLeftArrowScrollBehaviourIscorrect() { | |
| fun testFragment_enableOnboardingV2_rtl_checkLeftArrowScrollBehaviourIsCorrect() { |
Ditto above. Though perhaps we can be more specific than 'correct'?
There was a problem hiding this comment.
I updated name to testFragment_enableOnboardingV2_rtl_checkLeftArrowScrollsListToTheRight, ditto for similar tests.
| ) | ||
| ) | ||
|
|
||
| testCoroutineDispatchers.runCurrent() |
There was a problem hiding this comment.
Why is this needed? Can't the test just end?
Ditto below.
There was a problem hiding this comment.
It is not required. I have removed it everywhere.
| recyclerViewId = R.id.profiles_list_landscape, | ||
| position = 0, | ||
| targetViewId = R.id.profile_name_text, | ||
| itemMatcher = allOf(withText("Admin"), isDisplayed()) |
There was a problem hiding this comment.
This doesn't technically verify that it's alphabetical, just that admin is first. Do we already verify elsewhere the whole list is populated correctly?
There was a problem hiding this comment.
I have refactored the test to verify the position of all the created profiles.
| @@ -100,7 +100,8 @@ | |||
| android:layout_width="@dimen/profile_selection_fragment_icon_size" | |||
| android:layout_height="@dimen/profile_selection_fragment_icon_size" | |||
| android:layout_marginEnd="@dimen/profile_selection_fragment_add_profile_button_margin_end" | |||
| android:contentDescription="@string/profile_selection_profile_icon_description" | |||
| android:layout_marginBottom="@dimen/profile_selection_fragment_add_button_margin_bottom" | |||
| android:contentDescription="@string/profile_selection_profile_icon_description" | |||
There was a problem hiding this comment.
Nit: please remove extra indentation here.
| * Used to retrieve the layout direction that should be used to mirror the direction of the | ||
| * list based on locale. | ||
| */ | ||
| @Inject lateinit var resourceHandler: AppLanguageResourceHandler |
There was a problem hiding this comment.
Since this class is a presenter, this can be injected directly via the class's constructor.
| @@ -1225,7 +1226,7 @@ class ProfileManagementController @Inject constructor( | |||
| // TODO(#3616): Migrate to the proper SDK 29+ APIs. | |||
| @Suppress("DEPRECATION") // The code is correct for targeted versions of Android. | |||
| val bitmap = MediaStore.Images.Media.getBitmap(context.contentResolver, avatarImagePath) | |||
| val fileName = avatarImagePath.pathSegments.last() | |||
| val fileName = UUID.randomUUID().toString() | |||
There was a problem hiding this comment.
Are you able to document what you've tried and what hasn't worked in a new issue that we could file to track this? While I'm fine with postponing adding a test for this now, it will eventually become a requirement as we work in the long-term toward 100% code coverage. It'd be nice to capture your efforts so that they don't need to be redone in the future.
| val offset = | ||
| if (isLeft) scrollDistance - scrollableWidth else scrollableWidth - scrollDistance |
There was a problem hiding this comment.
Sorry, I don't think that quite answers my question. The video seems to start with the list of profiles scrolled in the middle, so it's unclear which side of the list the user begins on. Is it the right or left when in RTL?
APK & AAB differences analysisNote that this is a summarized snapshot. See the CI artifacts for detailed differences. DevExpand to see flavor specificsUniversal APKAPK file size: 19 MiB (old), 19 MiB (new), 43 KiB (Added) APK download size (estimated): 17 MiB (old), 17 MiB (new), 34 KiB (Added) Method count: 260321 (old), 260615 (new), 294 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 6830 (old), 6887 (new), 57 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 18 MiB (old), 18 MiB (new), 42 KiB (Added)
Configuration hdpiAPK file size: 50 KiB (old), 50 KiB (new), 60 bytes (Added) Configuration ldpiAPK file size: 49 KiB (old), 49 KiB (new), 48 bytes (Added) Configuration mdpiAPK file size: 45 KiB (old), 45 KiB (new), 48 bytes (Added) Configuration tvdpiAPK file size: 86 KiB (old), 86 KiB (new), 84 bytes (Added) Configuration xhdpiAPK file size: 57 KiB (old), 57 KiB (new), 60 bytes (Added) Configuration xxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 72 bytes (Added) Configuration xxxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 84 bytes (Added) AlphaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 37 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 26 KiB (Added) Method count: 115840 (old), 116029 (new), 189 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5798 (old), 5846 (new), 48 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) BetaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 31 KiB (Added) Method count: 115846 (old), 116035 (new), 189 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5798 (old), 5846 (new), 48 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 11 MiB (new), 35 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) GaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 31 KiB (Added) Method count: 115846 (old), 116035 (new), 189 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5798 (old), 5846 (new), 48 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 11 MiB (new), 35 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) |
- adjust profile list header margin top. - fix exit profile for the sole learner in classroom activity.
Admin profile exits back to PIN activity. Expected destination is the profile chooser. Fixed by finishing pin activity after login.
When a profile was deleted, the span count remained set to multiple profiles because of the use of the WasProfileEverAdded property. A single profile should always be center aligned.
Coverage ReportResultsNumber of files assessed: 52 Passing coverageFiles with passing code coverage
Exempted coverageFiles exempted from coverage
|
I reduced the end margin to create more space between the recyclerview and the button. Also increased the spacing between the button and the prompt.
The single profile(or first few profiles), are now placed from the 1/3 part of the screen.
The layout works as expected both with the arrows and with regular scrolling. |
APK & AAB differences analysisNote that this is a summarized snapshot. See the CI artifacts for detailed differences. DevExpand to see flavor specificsUniversal APKAPK file size: 19 MiB (old), 19 MiB (new), 41 KiB (Added) APK download size (estimated): 17 MiB (old), 17 MiB (new), 32 KiB (Added) Method count: 260308 (old), 260600 (new), 292 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 6832 (old), 6887 (new), 55 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 18 MiB (old), 18 MiB (new), 40 KiB (Added)
Configuration hdpiAPK file size: 50 KiB (old), 50 KiB (new), 60 bytes (Added) Configuration ldpiAPK file size: 49 KiB (old), 49 KiB (new), 48 bytes (Added) Configuration mdpiAPK file size: 45 KiB (old), 45 KiB (new), 48 bytes (Added) Configuration tvdpiAPK file size: 86 KiB (old), 86 KiB (new), 84 bytes (Added) Configuration xhdpiAPK file size: 57 KiB (old), 57 KiB (new), 60 bytes (Added) Configuration xxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 72 bytes (Added) Configuration xxxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 84 bytes (Added) AlphaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 26 KiB (Added) Method count: 115790 (old), 116020 (new), 230 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5800 (old), 5846 (new), 46 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 11 MiB (old), 11 MiB (new), 35 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) BetaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 32 KiB (Added) Method count: 115796 (old), 116026 (new), 230 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5800 (old), 5846 (new), 46 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 11 MiB (new), 36 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) GaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 33 KiB (Added) Method count: 115796 (old), 116026 (new), 230 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5800 (old), 5846 (new), 46 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 11 MiB (new), 36 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) |
|
Hi @adhiamboperes, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 7 days, it will be automatically closed so that others can take up the issue. |
oppiabot
bot
added
the
stale
oppiabot
bot
removed
the
stale
APK & AAB differences analysisNote that this is a summarized snapshot. See the CI artifacts for detailed differences. DevExpand to see flavor specificsUniversal APKAPK file size: 19 MiB (old), 19 MiB (new), 42 KiB (Added) APK download size (estimated): 17 MiB (old), 17 MiB (new), 33 KiB (Added) Method count: 260308 (old), 260587 (new), 279 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 6832 (old), 6889 (new), 57 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 18 MiB (old), 18 MiB (new), 41 KiB (Added)
Configuration hdpiAPK file size: 50 KiB (old), 50 KiB (new), 60 bytes (Added) Configuration ldpiAPK file size: 49 KiB (old), 49 KiB (new), 48 bytes (Added) Configuration mdpiAPK file size: 45 KiB (old), 45 KiB (new), 48 bytes (Added) Configuration tvdpiAPK file size: 86 KiB (old), 86 KiB (new), 84 bytes (Added) Configuration xhdpiAPK file size: 57 KiB (old), 57 KiB (new), 60 bytes (Added) Configuration xxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 72 bytes (Added) Configuration xxxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 84 bytes (Added) AlphaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 29 KiB (Added) Method count: 115790 (old), 115970 (new), 180 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5800 (old), 5848 (new), 48 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 11 MiB (old), 11 MiB (new), 35 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) BetaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 32 KiB (Added) Method count: 115796 (old), 115976 (new), 180 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5800 (old), 5848 (new), 48 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 11 MiB (new), 35 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) GaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 36 KiB (Added) APK download size (estimated): 10 MiB (old), 10 MiB (new), 31 KiB (Added) Method count: 115796 (old), 115976 (new), 180 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5800 (old), 5848 (new), 48 (Added)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 11 MiB (new), 35 KiB (Added)
Configuration hdpiAPK file size: 43 KiB (old), 43 KiB (new), 24 bytes (Added) Configuration ldpiAPK file size: 44 KiB (old), 44 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 38 KiB (old), 38 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 73 KiB (old), 73 KiB (new), 24 bytes (Added) Configuration xhdpiAPK file size: 50 KiB (old), 50 KiB (new), 24 bytes (Added) Configuration xxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) Configuration xxxhdpiAPK file size: 55 KiB (old), 55 KiB (new), 24 bytes (Added) |
Explanation
Fixes #4072.
The selected image uri created by
com.miui.galleryapp in Xiaomi devices has a different format from the android recommended uri, and is therefore not parsed correctly. Changing the name of the stored file, to be generated using a random UUID, fixes the issue. Please see #4072 (comment) for more information. There have been no tests added for this bugfix because of issues setting up ShadowContentResolver to work with ProfileManagementController.Fixes Part of #4938: Creates the new Profile chooser UI.
The portrait mode layout is a grid layout recylerview, while landscape mode utilizes a custom recyclerview to create a carousel. This design choice is kind of inconsistent, but more intuitive for users in landscape mode.
ProfileChooserFragmentPresenterhandles the data binding and UI interactions for both layouts.Care has been taken to ensure the existing profile behaviour e.g. random color selection and 10 profiles limit have been retained.
Additionally, a new
ParentScreenenum has been created for the onboarding IntroActivity to control when to show the step count. Per PRD, the step count should not show for additional learners.There have been minor bugfixes to ensure all the user pathways work as detailed in the TDD.
Essential Checklist
For UI-specific PRs only
If your PR includes UI-related changes, then:
Phone
Tablet