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

font color problem #1741

Closed
TAforever opened this issue Oct 20, 2022 · 28 comments
Closed

font color problem #1741

TAforever opened this issue Oct 20, 2022 · 28 comments

Comments

@TAforever
Copy link

Today I compiled waybar-git and on most modules the color began to match the color of the background. Version 0.9.13 does not have this problem.
waybar-git

@TAforever
Copy link
Author

Here is version 0.9.13
waybar

@danielrparks
Copy link

This happens with the default style as well:
image

@frkli1965
Copy link

Related to issue #1726?

@danielrparks
Copy link

Looks like it's related. I bisected it to b8322c4 which is from the same PR referenced in that issue (#1120 ). Note that for me, this is affecting even non-custom modules.

@tpeacock19
Copy link

label:disabled,
button:disabled {
  color: inherit;
  background-image: none;
}

It seems like any module that does not have an "on-click" value in the config shows up as a disabled button. The above fixed this issue for me. Alternatively, you can add "on-click": "true", and it should pick up existing button styles.

@frkli1965
Copy link

Thanks tpeacock19 ...

Appending "on-click"; "true" to the config file helped to resolve the 'memory' module's font color for me whereas altering the 'style.css' with the provided suggestions didn't.

Decided to drop the "custom-menu" module as it's no longer supported it seems.

Kind regards,

@LudovicoPiero
Copy link

label:disabled,
button:disabled {
  color: inherit;
  background-image: none;
}

It seems like any module that does not have an "on-click" value in the config shows up as a disabled button. The above fixed this issue for me. Alternatively, you can add "on-click": "true", and it should pick up existing button styles.

This fixed my issue 👍

@Alexays
Copy link
Owner

Alexays commented Oct 27, 2022

@pinselimo Can we add something like in #1738 to handle this issue?

@bklaase
Copy link

bklaase commented Oct 28, 2022

label:disabled,
button:disabled {
  color: inherit;
  background-image: none;
}

It seems like any module that does not have an "on-click" value in the config shows up as a disabled button. The above fixed this issue for me. Alternatively, you can add "on-click": "true", and it should pick up existing button styles.

that improves it, but the colors are still muted. As if maybe opacity is lowered as well

@tpeacock19
Copy link

I've found a more robust solution that may work for you. You'll have to add all: unset as the first declaration in these two rules:

button {
  all: unset;
  ...[other css declarations]...
}
label {
  all: unset;
  ...[other css declarations]...
}

The all: unset rules seems to put everything back to default. It should cover all variations between themes.

@Alexays
Copy link
Owner

Alexays commented Nov 3, 2022

3cf027f Should fix the pb
Can someone try?

@TAforever
Copy link
Author

3cf027f Should fix the pb Can someone try?

The change happened, but still there is an error.
waybar_grim

@hajosattila
Copy link

Same problem here. Waybar 0.9.13 vs 0.9.15

scrn-2022-11-03-14-52-19

scrn-2022-11-03-14-52-09

  • Arch
  • Sway WM

@danielrparks
Copy link

danielrparks commented Nov 3, 2022

I'm still having the same problem with the default style as well.
image

@Alexays
Copy link
Owner

Alexays commented Nov 3, 2022

And with 92cc01f?

@ChrisLane
Copy link

And with 92cc01f?

Just installed git package at that point in history. Seems to be fixed.

@danielrparks
Copy link

I still have the same problem with 92cc01f .

@TAforever
Copy link
Author

And with 92cc01f?

Returned to the previous state.
new_waybar_grim

@Alexays
Copy link
Owner

Alexays commented Nov 4, 2022

And with latest one 3030850?
Sorry for the mess..

@roland-5
Copy link

roland-5 commented Nov 4, 2022

And with latest one 3030850? Sorry for the mess..

Still grey.
ps_20221104085308

@roland-5
Copy link

roland-5 commented Nov 4, 2022

So with last commit, battery icon changed and with it, it's color to white. :P

ps_20221104220740

@danielrparks
Copy link

The latest commit didn't work for me either.

@TAforever
Copy link
Author

For the sake of interest, I decided to launch river with waybar in BSPWM and there is no such problem there. Maybe this information will be useful to you.
river_X

@TAforever
Copy link
Author

And so: by experience, I realized that the problem lies in the GTK theme. For some reason, the font color started to depend on the theme. I installed the Gruvbox-Material-Dark theme instead of Qogir-dark as I have in BSPWM and there is no problem with the font.
20221106_17h40m58s_grim

@tpeacock19
Copy link

And so: by experience, I realized that the problem lies in the GTK theme.

Yes, I'm not sure there can be a solution implemented by Waybar since I imagine the theme will be applied after its loaded. The all: unset should be in the default configuration and perhaps highlighted in the wiki as a fix for this issue.

@TAforever
Copy link
Author

And so: by experience, I realized that the problem lies in the GTK theme.

Yes, I'm not sure there can be a solution implemented by Waybar since I imagine the theme will be applied after its loaded. The all: unset should be in the default configuration and perhaps highlighted in the wiki as a fix for this issue.

@Alexays this seems to be the solution to the problem.

@danielrparks
Copy link

Indeed, adding

* {
    all: unset;
}

to my configuration causes it to start working at 92cc01f.

Sorry about that! It didn't occur to me that my gtk theme would be applied after waybar's style changes.

ralgar added a commit to ralgar/dotfiles that referenced this issue Nov 11, 2022
ralgar added a commit to ralgar/dotfiles that referenced this issue Nov 11, 2022
Lyndeno added a commit to Lyndeno/nix-config that referenced this issue Nov 16, 2022
@Alexays
Copy link
Owner

Alexays commented Nov 24, 2022

Closed as reverted

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

No branches or pull requests

10 participants