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

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

Open
2 tasks done
tewarig opened this issue Jun 6, 2023 · 5 comments
Open
2 tasks done
Labels
bug Something isn't working

Comments

@tewarig
Copy link

tewarig commented Jun 6, 2023

👟 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.

Screen.Recording.2023-06-07.at.1.08.58.AM.mov

👎 Actual Behavior

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

Screen.Recording.2023-06-07.at.1.07.35.AM.mov

🎲 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?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@tewarig tewarig added the bug Something isn't working label Jun 6, 2023
@tewarig
Copy link
Author

tewarig commented Jun 8, 2023

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

priyanshuverma-dev pushed a commit to priyanshuverma-dev/pink that referenced this issue Jun 14, 2023
issue: Change cursor pointer in checkbox's disabled state appwrite#91

Edited: _radio-and-checkbox.scss
added cursor: default; in disabled state.
@joeyouss
Copy link

Thank you for opening this!

@Isha988
Copy link

Isha988 commented Jun 20, 2023

I want to work on this issue.

@ANKITSINGH065
Copy link
Contributor

please review @tewarig

@elad2412
Copy link
Contributor

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; }

TorstenDittmann added a commit that referenced this issue Nov 16, 2023
…r-in-checkbox's-disabled-state

Fix [Bug]: Change cursor pointer in checkbox's disabled state #91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants