Skip to content
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

Merged
merged 1 commit into from
Sep 5, 2021

Conversation

sauravrao637
Copy link
Contributor

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

added night value in res for Opening Theme
added splash_background_night

Before/After Screenshots/Screen Record

  • Before: Bright launch screen even when using dark mode system wide
  • After: Now it's darker :)

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

@sauravrao637
Copy link
Contributor Author

@opusforlife2 how does the darker splash screen looks?

@triallax triallax added the GUI Issue is related to the graphical user interface label Sep 4, 2021
@opusforlife2
Copy link
Collaborator

@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.)

@sauravrao637
Copy link
Contributor Author

@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?

@tsiflimagas
Copy link
Contributor

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?).

@triallax
Copy link
Contributor

triallax commented Sep 5, 2021

@tsiflimagas that's not possible unfortunately, see #6244 (comment).

@tsiflimagas
Copy link
Contributor

@mhmdanas oh, I see, thank you. No problem then :)

@opusforlife2
Copy link
Collaborator

@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

@TobiGr TobiGr merged commit 22c5135 into TeamNewPipe:dev Sep 5, 2021
@opusforlife2 opusforlife2 changed the title Added night variant for splash_background.xml Added darker splash screen for OS dark theme Sep 5, 2021
@opusforlife2
Copy link
Collaborator

@sauravrao637 You put the code changes as the PR title again. :P

Edited.

This was referenced Sep 5, 2021
@litetex litetex mentioned this pull request Sep 9, 2021
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI Issue is related to the graphical user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Darker splash screen for system wide night mode
5 participants