Skip to content
This repository has been archived by the owner on Mar 23, 2023. It is now read-only.

[FEATURE] Add Dark mode #209 #98 #31 #229

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

timfrankentwf
Copy link

@timfrankentwf timfrankentwf commented Sep 24, 2021

Let me start by saying that this is my very first Pull Request on a larger project, so sorry if I miss some obvious steps.

I enjoy having dark mode for everything and this application should not be left out.
This feature is related to #209, #98 and #31.
There I already mentioned that I used the dark theme from https://github.com/AngryCarrot789/WPFDarkTheme as I was doing pretty much the same thing.

Things that work:

  • Having a Dark Mode.
  • A checkbox to switch between Light and Dark mode.
  • Remember which mode was selected on the next startup (using the existing Settings class).
  • Video player and other dialogs also change mode.
  • The switch between light and dark theme is now handled before loading the window, preventing the visual artifact during startup.

afbeelding

@robvdpol
Copy link
Owner

Cool, I'll review this when I have the time.

@C0Nd3Mnd
Copy link

C0Nd3Mnd commented Jan 3, 2022

Would it be possible to inherit the dark mode setting from the system (Windows 10 or higher) instead of (or in addition to) having the setting in the app itself?

@Psybertr0n
Copy link

@robvdpol Can you merge @timfrankentwf's changes for Dark Mode for a release? It would be greatly appreciated. A setting to choose from Light, Dark and Inherit from System as suggested above would be nice.

@ForthtuN
Copy link

Yes, merge would be great, the white is hurting my eyes when watching at night. Dark theme would make this already amazing and nr1 app even better

@timfrankentwf
Copy link
Author

I have updated the pull request to fix some small issues.

@Tanguille
Copy link

Nice effort! I would love to see darkmode everywhere too :) Is it in the latest release? If not when could we expect it to release?

@ForthtuN
Copy link

Seems it has some new conflicts before, its not in the latest version atleast

@timfrankentwf
Copy link
Author

timfrankentwf commented Mar 28, 2022

The merge conflicts happen because it's behind the main branch, the issues are easy to solve (mostly allowing to add the few additional lines of code). But there's no point in re-syncing and fixing those issues again when new commits are still added to the main branch.

I have no control on if/when this feature will be added.

@T0m0rr0w
Copy link

T0m0rr0w commented Apr 9, 2022

Any progress on this? It's been over half a year and still no dark mode. It does not have to be perfect. Even if some UI elements require further patches atleast the main window would not hurt my eyes. From what i've seen most programs that add dark mode alse require patches down the line to presumably fix small UI issues.

It coule be labeled as beta or testing to let the user know that it is still work in progress.

@robvdpol
Copy link
Owner

Fixing the live streams (DRM) is top priority at the moment. Dark mode will come after that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants