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

CE-294 Make the love and favorite icons colorblind friendly #7137

Closed
DNin01 opened this issue Oct 24, 2022 · 5 comments
Closed

CE-294 Make the love and favorite icons colorblind friendly #7137

DNin01 opened this issue Oct 24, 2022 · 5 comments

Comments

@DNin01
Copy link

DNin01 commented Oct 24, 2022

The problem

Currently, colorblind people can have a hard time telling if they have loved/favorited a project because there isn't a major or consistent brightness difference between the inactive and active love/favorite buttons. See this comparison of the buttons when you haven't clicked them and when you have:
A comparison between the love and favorite icons when you haven't clicked them and when you have clicked them
I grayscaled this image and the loved icon was ever so slightly darker than the unloved icon and the favorited icon was a bit brighter than the unfavorited icon.
The same image but in black and white

The solution

Make the inactive love/favorite icons an outline and fill them in when active.
The top two icons have only a border. The next two icons are filled and colored in.

@edelfino11
Copy link
Contributor

Hi @DNin01 @benjiwheeler I'm a computer science student, and we are currently working towards contributing to an open source GitHub for one of our classes. I would love to take this issue and work towards solving it! Is it ok if I take this issue to work on it? I'm not sure if anyone else is already assigned to this issue, so asking just in case.

@DNin01
Copy link
Author

DNin01 commented Nov 9, 2022

Is it ok if I take this issue to work on it? I'm not sure if anyone else is already assigned to this issue, so asking just in case.

@edelfino11 Don't mind if you do 🙂

@benjiwheeler
Copy link
Contributor

@edelfino11 Thanks so much for offering to do this change! We're checking internally with some folks working on accessibility design, and we'll get back to you!

@edelfino11
Copy link
Contributor

No problem! @benjiwheeler I figure it might be fixed by simply not filling the svg graphic with the gray color. Also, do you happen to know how to make the main page load properly when I run it locally? When I npm start the repo locally, it looks like this:

Screen Shot 2022-11-27 at 2 24 51 PM

So I can't check if the change I made fixes the problem. If there's a slack, I can also ask there instead! Sorry for the inconvenience!

@DNin01
Copy link
Author

DNin01 commented Jan 31, 2023

Fixed by #7264.

@DNin01 DNin01 closed this as completed Jan 31, 2023
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

3 participants