You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Don't use cursor: pointer for buttons by default (#8962)
* Don't use `cursor: pointer` for buttons by default
This is a pretty common expectation but as outlined in in #8961 isn't really right. We considered this a long time ago but it felt too against the grain at the time. These days though very UI-forward applications like [Linear](https://linear.app/) are using the default cursor for buttons and I think this trend will continue as more people become aware that `cursor: pointer` is meant for links.
Let's update our defaults here to help nudge people in this direction and make it more common. If people want to change this in their own apps, it's just a line or two of CSS to add to their projects.
* Update changelog
Co-authored-by: Adam Wathan <[email protected]>
Copy file name to clipboardExpand all lines: src/css/preflight.css
-16Lines changed: 0 additions & 16 deletions
Original file line number
Diff line number
Diff line change
@@ -315,22 +315,6 @@ textarea::placeholder {
315
315
color:theme('colors.gray.400',#9ca3af); /* 2 */
316
316
}
317
317
318
-
/*
319
-
Set the default cursor for buttons.
320
-
*/
321
-
322
-
button,
323
-
[role="button"] {
324
-
cursor: pointer;
325
-
}
326
-
327
-
/*
328
-
Make sure disabled buttons don't get the pointer cursor.
329
-
*/
330
-
:disabled {
331
-
cursor: default;
332
-
}
333
-
334
318
/*
335
319
1. Make replaced elements `display: block` by default. (https://github.com/mozdevs/cssremedy/issues/14)
336
320
2. Add `vertical-align: middle` to align replaced elements more sensibly by default. (https://github.com/jensimmons/cssremedy/issues/14#issuecomment-634934210)
0 commit comments