twbs / bootstrap

The most popular HTML, CSS, and JavaScript framework for developing responsive, mobile first projects on the web.
https://getbootstrap.com
MIT License
169.94k stars 78.74k forks source link

Not Allowed Cursor Missing on Disabled Inputs #28181

Closed pixelshaded closed 5 years ago

pixelshaded commented 5 years ago

https://getbootstrap.com/docs/4.2/components/forms/#default-stacked states

Disabled checkboxes and radios are supported, but to provide a not-allowed cursor on hover of the parent

All of these examples have a disabled attribute, yet none of them have the not-allowed cursor. I've tried firefox and chrome. Also not sure what it means by parent label. Aren't all the labels in the example siblings of the input rather than a parent? I'm wondering if this meant putting disabled on the .form-check since this is the parent of both the label and the input.

OS: Windows 10 Enterprise (Build 16299.666) Chrome: 72.0.3626.81 Firefox Quantum: 64.0.2

GoRateInfo commented 5 years ago

Currently the disabled attribute isn't widely supported. If you right click the disabled element and add the css cursor: not-allowed; attribute to the example I believe it will give you the behavior you would want.

With this being the case I would assume you would need to apply the css cursor: not-allowed; until the disabled attribute is more widely adopted.

patrickhlauke commented 5 years ago

the shorter answer is: we removed cursor: not-allowed ages ago, but missed this reference in the docs

https://github.com/twbs/bootstrap/commit/7056f702408a97f01cb32bc6075d88ec27cdcf52

patrickhlauke commented 5 years ago

can't find the original discussion on this, but basically the rationale is along the same lines as here https://github.com/twbs/bootstrap/pull/23485#pullrequestreview-56624098