-
Notifications
You must be signed in to change notification settings - Fork 551
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
Default colors in ASAP7 are too dark when used on black #3165
Comments
I've also noticed that the lowest layer color (dark blue / RGB(0, 0, 254)) is hard to see, eg the preferred/non preferred tracks. Perhaps the color palette used is more suited to a white background. OpenROAD/src/gui/src/displayControls.cpp Line 1726 in 9f3dc03
|
I see that http://vrl.cs.brown.edu/color has changed some since I generated those colors (more knobs). I don't get the same set anymore fwiw. I'm open to suggestions. The first three layers have traditional values and I wouldn't change them too far. |
@maliberty I would prefer to preserve the color of the text to match that of the layer, so removing the alpha might be a good solution |
Indifferent |
@antonblanchard you might try setting the alpha=255 for m1 and see if that helps with tracks. |
@maliberty is this still a thing? |
Yes, still hard to see. |
Do you find it hard to see other shapes on that layer or just the text? |
Both. |
If this a general color issue and not a text issue that's somewhat different. You can change the color by double clicking the color chip. |
The feature request here is that openroad ships with color scheme(s) that don't have to be tweaked to get something readable. |
I tried http://vrl.cs.brown.edu/color with How does that seem? Perhaps you can play around and see if you find a palette that works for you. |
Readability is subjective - I find it does that already. You need to be more specific/helpful to get anything to happen. |
Pointer to OpenROAD code where this is stored? |
I'll try to make it quantitative: running a poll now. |
OpenROAD/src/gui/src/displayControls.cpp Line 1912 in b972b34
|
@maliberty Quick poll by 3 colleagues no-one thought it was easy, 2x hard to read, 1x readable but could be brighted. |
I already gave a suggestion above but got no response. |
I made a PR... #5982 |
fixes The-OpenROAD-Project#3165 Signed-off-by: Øyvind Harboe <[email protected]>
fixes The-OpenROAD-Project#3165 Signed-off-by: Øyvind Harboe <[email protected]>
fixes The-OpenROAD-Project#3165 Signed-off-by: Øyvind Harboe <[email protected]>
fixes The-OpenROAD-Project#3165 Co-authored-by: Peter Gadfort <[email protected]> Signed-off-by: Øyvind Harboe <[email protected]>
Description
The text against black for these pins is illegible.
Suggested Solution
Have a default color scheme that's better adapted to be readable against black background.
Additional Context
Of course it is possible to change the colors in the GUI, but this feature request is about readability in the default case without needing to configure things.
The text was updated successfully, but these errors were encountered: