tbleckert / react-select-search

⚡️ Lightweight select component for React
https://react-select-search.vercel.app
MIT License
678 stars 147 forks source link

Show "not-allowed" cursor on hover for disabled select input/disabled options [accessibility] #205

Closed govindsartaj closed 2 years ago

govindsartaj commented 3 years ago

Showing the "not-allowed" cursor when a user hovers on a disabled select input/disabled option might be more accessible for those with visual impairments/cannot distinguish between the different opacity levels. Thoughts?

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/tbleckert/react-select-search/HsyjFzUn6U7CtFhkcme61jmjrt4q
✅ Preview: https://react-select-search-git-fork-govindsartaj-curs-d0c716-tbleckert.vercel.app

tbleckert commented 2 years ago

Nice!

tbleckert commented 2 years ago

Could you revert the package-lock that is causing the conflict? If so I can merge this @govindsartaj

vercel[bot] commented 2 years ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
react-select-search ✅ Ready (Inspect) Visit Preview May 7, 2022 at 4:27PM (UTC)
govindsartaj commented 2 years ago

@tbleckert I ruined this branch lol, making a new PR.

edit: see #233