-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Enable per-app language preferences for Android < 13 #12444
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
Conversation
7e313d1
to
48c5c4f
Compare
Is this ready for review? Related documentation (we should put that somewhere into the code): https://android-developers.googleblog.com/2022/11/per-app-language-preferences-part-1.html |
48c5c4f
to
893a227
Compare
Yes.
I updated it. |
26af421
to
9e14f93
Compare
* Use build constants when possible * Inline variables * Don't use var for normal-sized types (that way it's easier to review) * Split code into methods
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, did some minor cleanup
app/src/main/java/org/schabi/newpipe/settings/ContentSettingsFragment.java
Show resolved
Hide resolved
Successfully tested on:
|
What is it?
Description of the changes in your PR
assureCorrectAppLanguage
method has also been removed.Before/After Screenshots/Screen Record
Screen_recording_20250719_193520.mp4
Screen_recording_20250719_193555.mp4
Fixes the following issue(s)
Relies on the following changes
APK testing
The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR. You can find more info and a video demonstration on this wiki page.
Due diligence