-
Notifications
You must be signed in to change notification settings - Fork 521
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix #5406: Migrate keyboard hiding functionality #5463
Fix #5406: Migrate keyboard hiding functionality #5463
Conversation
Hi! @subhajitxyz Welcome to Oppia! Could you please follow the instructions here and sign the CLA Sheet to get started? You'll need to do this before we can accept your PR. Thanks! |
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), 344 bytes (Removed) APK download size (estimated): 17 MiB (old), 17 MiB (new), 99 bytes (Added) Method count: 258445 (old), 258446 (new), 1 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 6775 (old), 6775 (new), 0 (No change)
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), 344 bytes (Removed) Configuration hdpiAPK file size: 49 KiB (old), 49 KiB (new), 0 bytes (No change) Configuration ldpiAPK file size: 48 KiB (old), 48 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 45 KiB (old), 45 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 85 KiB (old), 85 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 56 KiB (old), 56 KiB (new), 0 bytes (No change) Configuration xxhdpiAPK file size: 62 KiB (old), 62 KiB (new), 0 bytes (No change) Configuration xxxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 0 bytes (No change) AlphaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 88 bytes (Removed) APK download size (estimated): 10 MiB (old), 10 MiB (new), 144 bytes (Removed) Method count: 114844 (old), 114845 (new), 1 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5745 (old), 5745 (new), 0 (No change)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 10 MiB (new), 88 bytes (Removed) 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: 72 KiB (old), 72 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 49 KiB (old), 49 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), 100 bytes (Removed) APK download size (estimated): 10 MiB (old), 10 MiB (new), 728 bytes (Added) Method count: 114850 (old), 114851 (new), 1 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5745 (old), 5745 (new), 0 (No change)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 10 MiB (new), 100 bytes (Removed) 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: 72 KiB (old), 72 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 49 KiB (old), 49 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), 60 bytes (Removed) APK download size (estimated): 10 MiB (old), 10 MiB (new), 844 bytes (Added) Method count: 114850 (old), 114851 (new), 1 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5745 (old), 5745 (new), 0 (No change)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 10 MiB (new), 60 bytes (Removed) 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: 72 KiB (old), 72 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 49 KiB (old), 49 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) |
Now i have signed CLA. |
Hi @subhajitxyz, thanks for creating a PR. Could you please update your PR according to the essential checklist? |
Thank you. I am updating.
…On Mon, 15 Jul 2024, 12:18 pm Vishwajith Shettigar, < ***@***.***> wrote:
Assigned #5463 <#5463> to
@subhajitxyz <https://github.com/subhajitxyz>.
—
Reply to this email directly, view it on GitHub
<#5463 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/BEUAZ2W3S6TSMFG3OOTI243ZMNWEVAVCNFSM6AAAAABK2EKPJSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGUYDAOBYGIZDINQ>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checked.
@subhajitxyz update title also. Take #5382 for reference. |
I have updated the title as per your reference . Please review and let me know if there are any additional adjustments needed. |
@subhajitxyz replace Fixes with Fix in title, and assign @adhiamboperes. |
I have updated the title by replacing Fixes with Fix and assigned the issue to @adhiamboperes. Please check and let me know if everything looks good now. |
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), 344 bytes (Removed) APK download size (estimated): 17 MiB (old), 17 MiB (new), 99 bytes (Added) Method count: 258445 (old), 258446 (new), 1 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 6775 (old), 6775 (new), 0 (No change)
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), 348 bytes (Removed) Configuration hdpiAPK file size: 49 KiB (old), 49 KiB (new), 0 bytes (No change) Configuration ldpiAPK file size: 48 KiB (old), 48 KiB (new), 0 bytes (No change) Configuration mdpiAPK file size: 45 KiB (old), 45 KiB (new), 0 bytes (No change) Configuration tvdpiAPK file size: 85 KiB (old), 85 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 56 KiB (old), 56 KiB (new), 0 bytes (No change) Configuration xxhdpiAPK file size: 62 KiB (old), 62 KiB (new), 0 bytes (No change) Configuration xxxhdpiAPK file size: 63 KiB (old), 63 KiB (new), 0 bytes (No change) AlphaExpand to see flavor specificsUniversal APKAPK file size: 11 MiB (old), 11 MiB (new), 88 bytes (Removed) APK download size (estimated): 10 MiB (old), 10 MiB (new), 137 bytes (Removed) Method count: 114844 (old), 114845 (new), 1 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5745 (old), 5745 (new), 0 (No change)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 10 MiB (new), 88 bytes (Removed) 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: 72 KiB (old), 72 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 49 KiB (old), 49 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), 100 bytes (Removed) APK download size (estimated): 10 MiB (old), 10 MiB (new), 724 bytes (Added) Method count: 114850 (old), 114851 (new), 1 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5745 (old), 5745 (new), 0 (No change)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 10 MiB (new), 96 bytes (Removed) 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: 72 KiB (old), 72 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 49 KiB (old), 49 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), 56 bytes (Removed) APK download size (estimated): 10 MiB (old), 10 MiB (new), 835 bytes (Added) Method count: 114850 (old), 114851 (new), 1 (Added) Features: 2 (old), 2 (new), 0 (No change) Permissions: 6 (old), 6 (new), 0 (No change) Resources: 5745 (old), 5745 (new), 0 (No change)
Lesson assets: 111 (old), 111 (new), 0 (No change) AAB differencesExpand to see AAB specificsSupported configurations:
Base APKAPK file size: 10 MiB (old), 10 MiB (new), 60 bytes (Removed) 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: 72 KiB (old), 72 KiB (new), 0 bytes (No change) Configuration xhdpiAPK file size: 49 KiB (old), 49 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) |
@theMr17, PTAL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @subhajitxyz! Had one minor comment, PTAL.
app/src/main/java/org/oppia/android/app/player/state/StateFragmentPresenter.kt
Outdated
Show resolved
Hide resolved
…mentPresenter.kt Co-authored-by: Mr. 17 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @subhajitxyz, this LGTM!
Assigning @adhiamboperes for code owner reviews. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @subhajitxyz!
Thanks @Vishwajith-Shettigar and @theMr17 as well!
Unassigning @adhiamboperes since they have already approved the PR. |
Hi @subhajitxyz, this PR is ready to be merged. Please address any remaining comments prior to merging, and feel free to ask someone to merge your PR once the CI checks pass and you're happy with it. Thanks! |
Thank you all for guiding me through my first contribution. |
Explanation
Fixes #5406
Essential Checklist