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

Support for Proxmox 7.4 (after release of "official" dark mode) - few issues with pvediscord dark #160

Open
accessiblepixel opened this issue Mar 31, 2023 · 8 comments

Comments

@accessiblepixel
Copy link

Greetings and salutations! :)

Proxmox seems to have added their own official darkmode now, but I'm not really a fan. I've gotten very used to how things look with PVEDiscordDark so I went to reinstall it to re-enable it, and it mostly works.

There are a few icons that don't seem to be changed (the node icon, and the storage icon - they're quite bright/white) and the 'proxmox' logo looks a bit weird (orange and black on grey).

The graphs now use the black from the official dark mode skin (when you enable 'dark' in the options) so it doesn't look too out of place, but it'd be nicer if it was the same grey it used to be, but if it's not something that's easy to change I can live with that.

Of course the installer says that it's not a supported version (which makes sense!), but I do hope that it can be fully supported again soon.

I'd like to say firstly thank you for making this project available, I've used it for quite some time (and my eyes thank you!) and really appreciate the work you've put into it so far.

Looking forward to your reply.

Kind regards,
Jessica

@djraw
Copy link

djraw commented Mar 31, 2023

One important thing to do with the new 7.4 pve, though it seems paradox - after applying this Discord theme you should set the webinterface to "light".

Most changes the script applies only change the former standard, the light theme.

@accessiblepixel
Copy link
Author

Thanks for the reply - I swapped back to light mode, uninstalled the theme, reinstalled and the icons and logo now looks as it should with PVEDiscordDark however my graphs are now white and the number/details on the axis are unreadable.
I'm guessing I'm doing something wrong?

@djraw
Copy link

djraw commented Mar 31, 2023

I'm guessing I'm doing something wrong?

No, you are doing everything right. Unfortunately the changes to the graphs don't work anymore with 7.4

So either you have light graphs but most icons etc. work, or dark graphs but many other issues.

Anyway, as the script states for now 7.4 is unsupported and this time it really is. ;)

Honestly I use the included dark mode for now and hope for an update to this script.. or for proxmox to include it as a selectable option.

@accessiblepixel
Copy link
Author

Totally! I know it's an unsupported option, but I personally feel for open-source software and what not the price of admission is filing issues when I can so that people know that there's an issue and that people are using whatever it is :)

I think the workaround for me would be having the dark mode with the slightly odd icons, since the graphs not being usable is a worse trade off just for some less bright icons, and I look forward to if/when there's a new fix for us that love this theme (I do like that they've included a darkmode by default now (it's a about time! 😋) but I don't like change and I really like the way PVEDiscordDark looks.

I'm willing to throw in a couple of dollars to aid development of something that I basically use every day if that will help 😃 (but don't see an obvious way listed in the repo and not sure if the author(s) would like donations anyway).

Kind regards,
Jessica

@sterzy
Copy link

sterzy commented Mar 31, 2023

just to throw in my two cents, at least for charts adapting this theme would be fairly easy, you can simply set a couple css variables now, and the chart and gauge components will take care of the rest. this even works with a prefers-color-scheme css media query, as the components now listen on changes here.

this and some other changes should obsolete the PVEDiscordDark.js afaict.

@Weilbyte
Copy link
Owner

Really glad to hear how much you like the theme; as for donations, I am not accepting any at the moment and do not plan to. I still have yet to take a proper look at the new version, but it does seem promising that the charts now get their colors from CSS variables. I am not sure how the project would continue with future versions though since the inclusion of a built-in dark mode, as any further changes would require both compatibility with the current version and also the previous ones (but for those I am thinking the releases would be fine as no further changes are needed); hopefully we can figure something out.

@r4yzs
Copy link

r4yzs commented Apr 2, 2023

Hello,

This new dark theme is very ugly.

I hope a new version of this discord theme...

@AntonUden
Copy link

AntonUden commented Apr 20, 2023

setting the proxmox theme to light fixes the icons but makes the graphs have a white backgrounds instead

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

No branches or pull requests

6 participants