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

Custom toolbars no longer work with the new dark theme #202

Closed
Vulgaris-Viridis opened this issue Jan 2, 2020 · 5 comments
Closed

Custom toolbars no longer work with the new dark theme #202

Vulgaris-Viridis opened this issue Jan 2, 2020 · 5 comments

Comments

@Vulgaris-Viridis
Copy link

Vulgaris-Viridis commented Jan 2, 2020

@clsid2 please allow for some time before closing the issue, as helpful as the yesterday @adipose's answer was, (since it's maybe pointing in the right direction) the issue is however not solved.

This is what the toolbar_dark looks in .bmp
Toolbar_bmp

And this is what it looks like in .png
Toolbar_png

This is what it looks like with white theme, when rendered properly.
Toolbar_white

@adipose
Copy link

adipose commented Jan 2, 2020

Do you have a link to the toolbar file?

Not all toolbars will be compatible. It tries to invert the colors, but a multi colored toolbar may not work. Lots of custom toolbars are working, but I can't guarantee they all will.

@adipose
Copy link

adipose commented Jan 2, 2020

Also difference between bmp and png makes no sense, unless you lost the alpha channel when converting.

@Vulgaris-Viridis
Copy link
Author

Do you have a link to the toolbar file?

Sure, here you go:
https://trac.mpc-hc.org/raw-attachment/wiki/Toolbar_images/mpc_toolbar_oxygen_white_32.zip

Not all toolbars will be compatible. It tries to invert the colors, but a multi colored toolbar may not work. Lots of custom toolbars are working, but I can't guarantee they all will.

I see... Well the dark theme is obviously worth sacrificing the look a little bit if it doesn't work but the thing is, I've been using this toolbar for the past couple of years and now with any other mpc looks just wrong if you know what i mean. So i'd like to keep it if possible.. Anyway, thanks for looking into this @adipose..

@adipose
Copy link

adipose commented Jan 2, 2020

@Vulgaris-Viridis There are special rules for colored toolbars (a recent change). On reflection, I think it is a bug that the dark theme doesn't entirely honor them. Here's a test build to fix that issue.

https://mega.nz/#!hUBTyKDA!Rcom2bHdu3_A7f1OA4IrWkT2EVWr19wVequb-pStxHA

Now you can name your toolbar color_toolbar_dark.bmp, and it won't mess with it. If you don't like it, tough--adjust the colors in photoshop. ;)

#207

@Vulgaris-Viridis
Copy link
Author

Hi @adipose, well first of I appreciate you taking the time to look into this, I've just tested it, it works great!

I feel there's no need to keep this issue open any longer, especially since the official fix is on it's way as well.

Once again, much appreciated!

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

2 participants