MediaElch main window stuck on bright white, not adjusting to KDE dark theme

[ioogithub]

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

1. Open MediaElch.
2. Observe that the left navigation bar, settings window, right click context menu all inherited the dark theme and look really nice.
3. Observe that the main window background and top navigation bar are drawn in a bright white/off white yellowish colour. It has not inherited the dark them as the other parts of the app have. There doesn't appear to be any setting to adjust the colour.

Expected behavior
I would expect that the app would be able to inherit the colors from the dark theme in all parts of the app.

MediaElch Version:

• [X ] 2.6.6 stable, AppImage version downloaded directly from website.

Operating System:

• [X ] Linux (distribution: KDE Neon 5.20 )

Additional context
Is the main window background actually hard coded to be this white colour? I am currently using the Arc Dark global theme. I can see that parts of MediaElch inherit the colour scheme perfectly and beautiful. The main window background however is such a contrast to every other app I have that when I open it it strains my eyes, I have to immediately adjust the screen brightness down. Then when I minimize the window I have to adjust the brightness back up for all my other apps/OS.

If Media Elch can inherit the colours properly for the settings, left navigation bar, right click context menus, progress windows and most other places,, can the main window background and top navigation bar also be fixed?

This is close to a perfect app for me and I have been searching for this for a long time however I won't be able to use it if the main window background is hard coded to be such a bright white.

[bugwelle]

Hi,

thanks for your feedback! MediaElch's main window has hard-coded colors... A while ago I moved all hard-coded styles to this CSS file: https://github.com/Komet/MediaElch/blob/master/src/ui/default.css

I would need to adjust that file. However, I currently do not have the time to do so. :-/
Only the main window uses custom colors and styles. All other windows are default Qt-windows which adapt KDE's color themes.

Dark-theme support for KDE has been added with v2.6.0 ( see https://mediaelch.github.io/mediaelch-blog/posts/mediaelch-v2.6.0/ ) but no further development has been done since then.

If other contributors are reading this: I would gladly review a dark theme CSS file!

Regards,
Andre

[ticao2]

And I, as a Windows user, remain very envious of Linux users, who have this Dark version of MediaElch.
How long, my God? :-)

[ioogithub]

A while ago I moved all hard-coded styles to this CSS file: https://github.com/Komet/MediaElch/blob/master/src/ui/default.css

I can't edit and change this file on my local installation can I?

[bugwelle]

No, at the moment the CSS file is part of the binary. If you like, I can add an option to advancedsettings.xml where you can specify a CSS file that MediaElch loads. Then you could modify the CSS file, restart MediaElch and see your changes.

Would that be acceptable to you? :-)

(I'm unsure whether a "reload window" is easy to implement but I could also look at that)

[ticao2]

Who knows, maybe this will facilitate the implementation of my old wish for an interface more built to be seen in a Media Center, on a TV with 56 ', about 4 or 5 meters away, larger characters and poster ...
https://i.imgur.com/hWRV7eM.jpg
But it is just a wish. I'm not a programmer. :-)

[ioogithub]

No, at the moment the CSS file is part of the binary. If you like, I can add an option to advancedsettings.xml where you can specify a CSS file that MediaElch loads. Then you could modify the CSS file, restart MediaElch and see your changes.

That sounds like a good solution.

[bugwelle]

Will be implemented with https://github.com/Komet/MediaElch/pull/1040

[bugwelle]

I've tested https://github.com/Komet/MediaElch/pull/1040 and have added screenshots there. :-)

Please note that there are still quite a few places with hard-coded colors. I have to fix them step by step.

[ioogithub]

Awesome response time! How can I test this?

[bugwelle]

You have to create an advancedsettings.xml file. Copy the contents from https://github.com/Komet/MediaElch/blob/master/docs/advancedsettings.xml into it.

(_Have a look at our documentation about advanced settings: https://mediaelch.github.io/mediaelch-doc/settings.html#advanced-settings_ )

Un-comment this line:

https://github.com/Komet/MediaElch/blob/831d3076946bcaecf4f07267e170e4bc829beea9/docs/advancedsettings.xml#L43

And add an absolute path to your copy of default.css to it (or a relative one where the path is relative to MediaElch.exe).
You can find MediaElch's version at:
https://github.com/Komet/MediaElch/blob/master/src/ui/default.css

To test that MediaElch actually uses the CSS file you can also remove all contents from the stylesheet and start MediaElch. It then should look like the screenshot from #1040. After that simply change the color values and see how it looks (you have to restart MediaElch after each change. I haven't implemented a "reload CSS file", yet).