clbr / radeontop

GNU General Public License v3.0
810 stars 72 forks source link

radeontop uses color 1 instead of background for the background color #134

Closed ghost closed 2 years ago

ghost commented 2 years ago

image

To find which color is actually used I have set my terminal color 1 to be pink in this screenshot.

Real use issue:

Usually color 1 is a medium grey on my system, which is great as a text color against the darker background, but does not work well for radeontop output.

image

As you can see this makes it pretty hard to read.

My current workaround is - after boot, I change the terminal color 1, start an instance of radeontop and let it run forever, change the terminal color 1 back, so that all new terminal instances have the proper colors.

Possible solution suggestions:

  1. Use terminal background color for the background of radeontop.
  2. Add a config file to overwrite the colors used (in terminal color code format if that makes it easier).
clbr commented 2 years ago

Use the transparency option? Sounds like what you're looking for, it was added for people with transparent terminals who wanted transparent bg.

Otherwise this is not the app's problem - you're overriding terminal colors, and as such, it's your and your terminal's issue. Radeontop sets the bg to black, not color 1. If you're overriding black to medium grey, well, that's your choice.

ghost commented 2 years ago

Sorry, I have looked at the manpage briefly but overlooked the transparency option and when looking through the closed issues, I didn't find anything color related either.

And you are right, in the default Breeze color configuration, Color 1 is indeed equal to the background color.