search

mdgriffith / style-elements

Create styles that don't mysteriously break!
http://package.elm-lang.org/packages/mdgriffith/style-elements/latest
BSD 3-Clause "New" or "Revised" License
445 stars 49 forks source link

Do not use a pointer cursor for the button elements by default #96

Closed smt116 closed 6 years ago

smt116 commented 7 years ago

Using an inline style for the cursor makes it impossible to set different value when needed without using !important. This is a case with, for example, disabled buttons which should have cursor: not-allowed style.

CSS specification for the cursor1 says that pointer value should be used for indicating a link. In many cases, the button selector have a different role than redirecting to another location.

mdgriffith commented 6 years ago

So, I fall on the side of "clickable things should have a pointer icon" part of that debate :)

I'm going to keep it as is, but thanks!

In v5 we don't render using inline styles anymore, so it might be easier to override if you do decide to go that route.