Skip to content

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