-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
[Estuary] Fix selected colours #24256
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Also did a colour blindness filter check in GIMP and has good visibility in all three.
Possible side issue not for this PR but related, I noticed in FileBrowser that selectedcolor is hardcoded to green. Not sure why that is but I suspect should be changed for consistency.
<selectedcolor>green</selectedcolor> |
@HitcherUK the point made about hardcoded green in FileBrowser, I think that should be addressed. Can you do that? if so can you add here or would you prefer to do a separate PR? |
I can add it to this PR as it covers 'selectedcolor'. |
Added fix for file browser selected colour being green. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like what I'm seeing in the screenshots!
LGTM. @HitcherUK @jurialmunkey Was looking to check out the FileBrowser change just to verify it picks up the default label |
@jjd-uk - Dialog class has an I assume that's what select colour is used -- though I'm not sure if I've ever seen anything use it. |
Description
Fixes the selected colours in lists for the default and charcoal schemes.
Motivation and context
It wasn't obvious what the selected items were in lists for the default and charcoal colour schemes. This fixes both of them.
How has this been tested?
Tested locally.
What is the effect on users?
Screenshots (if appropriate):
Before (default):
Before (charcoal):
Comparison with brown:
After (default):
After (charcoal):
Types of change
Checklist: