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

Text color with dark themes #640

Open
0xbilko opened this issue Jun 3, 2020 · 15 comments
Open

Text color with dark themes #640

0xbilko opened this issue Jun 3, 2020 · 15 comments
Milestone

Comments

@0xbilko
Copy link
Contributor

0xbilko commented Jun 3, 2020

The color of text for connections on dark themes is poorly read.
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. Setup any dark theme for gtk
  2. Run asbru-cm
  3. See text color in connections list

Screenshots
https://imgur.com/OkszR7x

Environment (please complete the following information):

@midniteslice
Copy link

I was just coming here with the same issue.
Version 6.2.0 on kubuntu 20.04 has the same issue. Version 6.1.3 on fedora 31 running kde does not have the issue.
I have tried many different themes with the same result.

@popxunga
Copy link

popxunga commented Jun 3, 2020

Hi ...

I do use Ad-waita Dark theme and it looks good.

My Asbru setup looks like this:

image

image

Let us know if that helps.

@0xbilko
Copy link
Contributor Author

0xbilko commented Jun 3, 2020

@popxunga , thanks, the "Look and Feel" setting helped me, but it would be better if the default settings were from the system theme.

@midniteslice
Copy link

That did it for me too. I agree with what beliys said about ti being default.
Thank You!

@popxunga
Copy link

popxunga commented Jun 3, 2020

Indeed ... the default colors used may need some adjustments ...

@gfrenoy
Copy link
Contributor

gfrenoy commented Jun 3, 2020

Not sure what you mean by the "default":

  • when you first start Ásbrú (so with a brand new configuration directory)
  • when you switch your GTK theme
  • when you switch your Ásbrú Icons theme

There is no magic, we need to define the logic that needs to be implemented and I don't see here what to do exactly.

@gfrenoy gfrenoy added the question Question about how to use Ásbrú or how to work with some specific connection type. label Jun 3, 2020
@popxunga
Copy link

popxunga commented Jun 3, 2020

Hi @gfrenoy,
From my point of view, the default text/icon colors should always be relative to the Asbru themes being used, only.
This may be more difficult as it looks on a 1st view, but as basic principle I would say that the logic behind this can be:
For the Asbru dark theme, light colors for text and icons should be used by default;
For the Default or Colored themes, dark colors for text and icons should be used by default.
If the user decides to use the System theme then is not an Asbru problem.

@0xbilko
Copy link
Contributor Author

0xbilko commented Jun 4, 2020

I agree with @popxunga , but when using a system theme, the colors should be taken from the theme that the system uses.

@popxunga
Copy link

popxunga commented Jun 4, 2020

Hi @beliys ... that's what I tried to meant, when the System theme is selected on Asbru, then Asbru should just pick up the colors defined in the theme.

@Mjolinir
Copy link

I agree with the gentlemen above, the colors, fonts and other theme related items should correspond to what the user has defined on the system by default. I like the idea of being able to override system defaults in the app, but the default should be what is set in the system. This allows for a consistent look and feel and makes the app feel more enterprise quality.

@gfrenoy gfrenoy added enhancement and removed question Question about how to use Ásbrú or how to work with some specific connection type. labels Jun 22, 2020
@gfrenoy
Copy link
Contributor

gfrenoy commented Jun 22, 2020

Understood, there's definitely something to improve here ...

The main issue I see here is that we do set a default color. Actually, we should have the option to "unset" the color. Which would mean that this would use the system color.

Sorry could not find a lot of time to improve Ásbrú recently ; let me put this into the priority for the next release so I do forget.

In the meantime, any kind of contribution will be of course very welcome.

@AlexFolland
Copy link

Setting the icon theme to asbru-dark did not fix the connection name text color for me, and I don't see why it should, since it is called an icon theme. Also, the "Save and Close" button in my interface no longer works after changing these settings, so I've been editing the yml file directly to change settings now, and nothing seems to have allowed the connection name text to become white. I will note that all other text looks great though. It's just that text in the connection list that's black on black.

@gfrenoy gfrenoy modified the milestones: 6.4.0, 6.4.1 Nov 6, 2022
@dennisTGC
Copy link

The setting does nothing on my system when asbru-cm is installed as a snap. I tried asbru-dark and system. I'm on asbru 6.4.0. Let me know if I can test any more.

I'm on kubuntu 22.04.1

@dennisTGC
Copy link

dennisTGC commented Nov 16, 2022

The setting does nothing on my system. I tried asbru-dark and system. I'm on asbru 6.4.1. Let me know if I can test any more.

I'm on kubuntu 22.04.1, asbrucm-loki 6.4.1, wayland.

@gfrenoy gfrenoy modified the milestones: 6.4.1, 6.5.0 Nov 16, 2022
@gfrenoy
Copy link
Contributor

gfrenoy commented Nov 16, 2022

This issue is related to a enhancement request that is not yet implemented. It is somehow expected that it is not working yet...

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

7 participants