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

[Carbon X] DatePicker color tokens applied with g90 and g100 themes have inaccessible color contrast #1710

Closed
jendowns opened this issue Jan 30, 2019 · 10 comments

Comments

@jendowns
Copy link
Contributor

jendowns commented Jan 30, 2019

Detailed description

A number of colors/tokens being applied to the DatePicker are not accessible when the g100 or g90 themes from @carbon/themes are used.

Here are some example screenshots...

g100 theme

Simple date picker

screen shot 2019-01-29 at 4 40 08 pm

Range with min and max

screen shot 2019-01-30 at 10 06 47 am

g90 theme

Range

screen shot 2019-01-30 at 10 01 51 am

@jendowns jendowns changed the title [Carbon X] DatePicker color tokens applied with g90 and g100 themes are inaccessible [Carbon X] DatePicker color tokens applied with g90 and g100 themes have inaccessible color contrast Jan 30, 2019
@jendowns
Copy link
Contributor Author

jendowns commented Jan 30, 2019

So this isn't a blocker for us right now, since I can override colors selectively in our addons library to make them accessible.

However, I really want to make sure I have the right intended color for the bottom border on the input element -- just so my override is in the right ballpark, at least. 🤔

@IBM/carbon-designers could anyone let me know what color that bottom input border should be for a g100 theme? Thank you 🙏 (EDIT: I'm referring to the bottom border color as shown below, before the input is select/focused/active)

screen shot 2019-01-30 at 11 27 02 am

@aagonzales
Copy link
Member

aagonzales commented Feb 1, 2019

  • The underline for the fields should be $ui-04 which is Gray 60 #6F6F6F in the Gray 100 theme
  • All the icons should be using the token $icon-01
  • The day ranges are just hard coded and will have to be over ridden. It seems excessive to make a token just for that. Could be convinced otherwise though. Could maybe be like a $highlight token or something, could maybe be used elsewhere
  • The placeholder text in the fields should be using $text-03 which should be Gray 60 #6F6F6F in the G100 Theme

@aagonzales
Copy link
Member

aagonzales commented Feb 1, 2019

What the Duo file says the dark date picker should look like (ignore their field styles):

  • highlight is Blue 90 #021677

image

@jendowns
Copy link
Contributor Author

I'm waiting for #1795 to be merged (as I need the new $highlight token from @carbon/themes) before I can proceed with proposing a fix for this 👍

@joshblack
Copy link
Contributor

@jendowns just a heads up, will pull out the updates into a two parts, I think. Hoping to have it done this morning if folks have a chance to leave feedback 👍

@jendowns
Copy link
Contributor Author

Sounds good -- thank you!

@jendowns
Copy link
Contributor Author

jendowns commented Feb 21, 2019

@aagonzales Quick question -- That "selected" color (the start/end selected date in a range) and the text + "dot" color for the current date look like blue60 in these dark themes. The token used throughout looks like $interactive-01, which unfortunately is always blue60 for all themes -- https://carbon-elements.netlify.com/themes/examples/preview/

Am I correct that the blue for select/today should be blue40 (or even blue50) instead? Do you think $interactive-01 needs to be modified... or a new token needs to be created?

@aagonzales
Copy link
Member

@jendowns we're one step a head of you this time. We just added in a new $interactive-04 that will those lighter blues in the dark themes but still keeping Blue 60 in the dark themes for the primary buttons with $interactive-01 https://github.com/IBM/carbon-elements/issues/366

@stale
Copy link

stale bot commented May 1, 2019

We've marked this issue as stale because there hasn't been any activity for a couple of weeks. If there's no further activity on this issue in the next three days then we'll close it. Thanks for your contributions.

@stale stale bot added the wontfix label May 1, 2019
@aagonzales
Copy link
Member

aagonzales commented May 2, 2019

I think this has been resolved with the addition of the $highlight and $interactive-04 color tokens.

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

4 participants