RaceControl icon indicating copy to clipboard operation
RaceControl copied to clipboard

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

Open timfrankentwf opened this issue 3 years ago • 10 comments

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:

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

afbeelding

timfrankentwf avatar Sep 24 '21 22:09 timfrankentwf

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

robvdpol avatar Sep 27 '21 07:09 robvdpol

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?

C0Nd3Mnd avatar Jan 03 '22 02:01 C0Nd3Mnd

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

Psybertr0n avatar Mar 10 '22 20:03 Psybertr0n

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

ForthtuN avatar Mar 11 '22 04:03 ForthtuN

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

timfrankentwf avatar Mar 12 '22 17:03 timfrankentwf

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?

Tanguille avatar Mar 15 '22 21:03 Tanguille

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

ForthtuN avatar Mar 27 '22 02:03 ForthtuN

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.

timfrankentwf avatar Mar 28 '22 18:03 timfrankentwf

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.

T0m0rr0w avatar Apr 09 '22 15:04 T0m0rr0w

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

robvdpol avatar Apr 11 '22 06:04 robvdpol