Merge pull request #330 from appwrite/fix-disabled-state

ItzNotABug · web-flow · commit c5c10b498427 · 2025-05-27T16:25:37.000+05:30
Fix disabled state on checkbox's hover
diff --git a/v2/pink-sb/src/lib/selector/Checkbox.svelte b/v2/pink-sb/src/lib/selector/Checkbox.svelte
@@ -96,7 +96,7 @@
             border-radius: var(--border-radius-xxs);
         }
 
-        &:hover:not(.active):not([aria-disabled='true']) {
+        &:hover:not(.active):not([aria-disabled='true']):not([disabled]) {
             background-color: var(--overlay-button-neutral-hover);
         }
 
@@ -105,9 +105,15 @@
             background-color: var(--bgcolor-neutral-invert);
         }
 
-        &:disabled {
+        &:disabled,
+        &[aria-disabled='true'] {
             opacity: 0.4;
+            cursor: default;
             background-color: var(--bgcolor-neutral-tertiary);
+
+            &:hover {
+                background-color: var(--bgcolor-neutral-tertiary);
+            }
         }
 
         &:focus,

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,7 @@`
`96`	`96`	`border-radius: var(--border-radius-xxs);`
`97`	`97`	`}`
`98`	`98`
`99`		`- &:hover:not(.active):not([aria-disabled='true']) {`
	`99`	`+ &:hover:not(.active):not([aria-disabled='true']):not([disabled]) {`
`100`	`100`	`background-color: var(--overlay-button-neutral-hover);`
`101`	`101`	`}`
`102`	`102`
`@@ -105,9 +105,15 @@`
`105`	`105`	`background-color: var(--bgcolor-neutral-invert);`
`106`	`106`	`}`
`107`	`107`
`108`		`- &:disabled {`
	`108`	`+ &:disabled,`
	`109`	`+ &[aria-disabled='true'] {`
`109`	`110`	`opacity: 0.4;`
	`111`	`+ cursor: default;`
`110`	`112`	`background-color: var(--bgcolor-neutral-tertiary);`
	`113`	`+`
	`114`	`+ &:hover {`
	`115`	`+ background-color: var(--bgcolor-neutral-tertiary);`
	`116`	`+ }`
`111`	`117`	`}`
`112`	`118`
`113`	`119`	`&:focus,`