-
-
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
Added darker splash screen for OS dark theme #7055
Conversation
@opusforlife2 how does the darker splash screen looks? |
@sauravrao637 Well, I would need before/after screenshots to compare. Please add them to the description, since this is a UI change. But if it's the same dark red colour as the app, then I think it would look great. That's the colour I was expecting the splash screen to be after your earlier PR. (Though I did fantasise for a moment about a dark shadowed Newpipe icon on a dark grey background, to be honest.) |
@opusforlife2 Yup it's the same dark color, we can make it black or dark grey as well , what do you think will be better? |
I like the colour, but is it possible to make it follow the app's theme (or if you intended to do it so, is it possible to fix it?). |
@tsiflimagas that's not possible unfortunately, see #6244 (comment). |
@mhmdanas oh, I see, thank you. No problem then :) |
@sauravrao637 I think the current colour is fine for this PR. If you have some spare time later, we could experiment with colours in a new issue. :D |
@sauravrao637 You put the code changes as the PR title again. :P Edited. |
What is it?
Description of the changes in your PR
added night value in res for Opening Theme
added splash_background_night
Before/After Screenshots/Screen Record
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.
Due diligence