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

Use default cursor on disabled checkboxes and radio buttons #4167

Merged
merged 1 commit into from Oct 12, 2023

Conversation

ivarnakken
Copy link
Member

Description

A pointer cursor indicates to the user that the input fields can be interacted with, and should therefore be changed accordingly when they are disabled.

Result

Cursor now behaves as expected.

Testing

  • I have thoroughly tested my changes.

Tested both input fields as both disabled fields and normal fields.

@ivarnakken ivarnakken added review-needed Pull requests that need review bug-fix Pull requests that fix a bug small-fix Pull requests that fix something small labels Oct 4, 2023
@ivarnakken ivarnakken requested a review from a team October 4, 2023 22:35
@ivarnakken ivarnakken self-assigned this Oct 4, 2023
Copy link
Contributor

@ollfkaih ollfkaih left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@ivarnakken ivarnakken force-pushed the checkbox-and-radiobuttons-disabled-cursor branch from 981e301 to 98da19e Compare October 6, 2023 12:19
@ivarnakken ivarnakken added approved Pull requests that have been approved ready-to-merge Pull requests that have been approved and are ready to be merged and removed review-needed Pull requests that need review labels Oct 6, 2023
A pointer cursor indicates to the user that the input fields can be interacted with, and should therefore be changed accordingly when they are disabled.
@ivarnakken ivarnakken force-pushed the checkbox-and-radiobuttons-disabled-cursor branch from 98da19e to fd3803d Compare October 12, 2023 11:00
@github-actions github-actions bot added the review-needed Pull requests that need review label Oct 12, 2023
@ivarnakken ivarnakken merged commit 439eabe into master Oct 12, 2023
4 checks passed
@ivarnakken ivarnakken deleted the checkbox-and-radiobuttons-disabled-cursor branch October 12, 2023 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved bug-fix Pull requests that fix a bug ready-to-merge Pull requests that have been approved and are ready to be merged review-needed Pull requests that need review small-fix Pull requests that fix something small
Projects
None yet
4 participants