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

feat: Improve Accessibility of Checkbox States #604

Closed
wants to merge 1 commit into from

Conversation

rishabdev2997
Copy link

Feat: Design of checkboxes in monochrome mode - proposal

#580 - added the design

This pull request addresses the accessibility concerns related to the current checkboxes in the accessibility menu. The existing checkboxes, represented as circles, do not clearly reflect their state in monochrome mode, making it difficult for users relying on non-color cues to differentiate between checked and unchecked states.

How to see the changes

  • Go to accessibility drop down
  • Click the down arrow to see the drop down options
  • Click the boxes to see the effects along with the tick check box

These modifications will enhance the user experience for those with visual impairments or those using monochrome displays, making the checkboxes more intuitive and accessible.

Here is the screenshot

p5js.mp4

@rishabdev2997
Copy link
Author

Hi @limzykenneth,

I've submitted a PR to enhance checkbox accessibility by replacing circles with tick marks for improved clarity in monochrome mode. I’d appreciate it if you could review the changes, and please let me know if any adjustments are needed.

Thank you!

@limzykenneth
Copy link
Member

I'm not sure inline SVG is necessarily the route we want to go with and we have not decided on what to go with. I would suggest furthering the discussions in the issue until a consensus is reached first. Thanks!

@rishabdev2997 rishabdev2997 closed this by deleting the head repository Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants