pink icon indicating copy to clipboard operation
pink copied to clipboard

🐛 Bug Report: Change cursor pointer in checkbox's disabled state

Open tewarig opened this issue 1 year ago • 5 comments

👟 Reproduction steps

hover on the top of the checkbox which is disabled. it will change the cursor to the pointer.

👍 Expected behavior

Ideally, since the checkbox is disabled we should not show a cursor pointer in this case. we can either set the cursor to not-allowed. or just remove the cursor pointer if the checkbox is disabled. here is an example of how it should work in a disabled state.

https://github.com/appwrite/pink/assets/54894783/375f9921-59a9-470b-a40e-8358eb5a9106

👎 Actual Behavior

hover on the top of the checkbox which is disabled. it will change the cursor to the pointer.

https://github.com/appwrite/pink/assets/54894783/7596e428-e45f-47c4-ab51-9f6740dce3e6

🎲 Appwrite version

Appwrite Cloud

💻 Operating system

MacOS

🧱 Your Environment

No response

👀 Have you spent some time to check if this issue has been raised before?

  • [X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

tewarig avatar Jun 06 '23 19:06 tewarig

one more thing this behaviour is same for switch , radio buttons in their disabled state. would love to work on this

tewarig avatar Jun 08 '23 10:06 tewarig

Thank you for opening this!

joeyouss avatar Jun 15 '23 19:06 joeyouss

I want to work on this issue.

Isha988 avatar Jun 20 '23 09:06 Isha988

please review @tewarig

ANKITSINGH065 avatar Jul 03 '23 09:07 ANKITSINGH065

We added a fix that removes the pointer to the initial cursor.

This will go in a future update of Pink design, which should be in the next two weeks.

Meanwhile, you can add a temporary fix in your local CSS if needed:

    input:disabled { cursor:initial; }

elad2412 avatar Jul 31 '23 17:07 elad2412