-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 Routeractivity theming #8332
Conversation
Why don't you remove all
What? |
These themes hides/makes the app transparent (e.g.
PR No2 is already work in progress to fix that 😆 |
Kudos, SonarCloud Quality Gate passed! |
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.
your last commit message is misleading, as it actually still is a static method :D
What is it?
Description of the changes in your PR
When the app is not running in the foreground and started via
RouterActivity
(e.g. clicking/sharing a link to NewPipe) incorrect things start to happens:Why does this happen?
color*
-attributes (colorPrimary
,colorPrimaryDark
, ...). I'm not sure why this was done in the first place, however it causes the theming issues on the UI.assureCorrectAppLanguage
andThemeHelper.setDayNightMode
. This causes further problems like the incorrect language or wrong colored theme elements.Before/After Screenshots/Screen Record
Done via the test manual below:
Before:
After:
Fixes the following issue(s)
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.
Test manual
Due diligence