891018 - CSS cursor property does not work properly on disabled inputs

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect, P2)

Description

[hmcgonig]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/27.0.1453.116 Safari/537.36

Steps to reproduce:

Apply any cursor style to a disabled input. I used cursor: not-allowed;

JS fiddle for an example: http://jsfiddle.net/MGxqP/1/


Actual results:

The cursor style only shows while hovering over the disabled input's padding, not while inside the actual input.


Expected results:

The whole input should display the new cursor style when hovered.

Comment 1

[hmcgonig]

This seems to be a regression, I just tried this in FF 19 and it works just fine.

Comment 2

[mjh563]

Basic testcase:
data:text/html,<input disabled style="cursor: not-allowed">

The regression range is

Last good nightly: 2013-02-16
First bad nightly: 2013-02-17

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=7bd555e2acfa&tochange=c4de6de47382

Comment 3

[mjh563]

Bug 841828 is the obvious one...

Comment 6

[Mats Palmgren (inactive)]

Mounir, repeating my question from duped bug 929988:

Why do we need the "> .anonymous-div" in those selectors?
Wouldn't this be sufficient:

input:disabled, textarea:disabled {
  cursor: default;
}

Comment 7

[Mounir Lamouri (:mounir)]

(In reply to Mats Palmgren (:mats) from comment #6)
> Mounir, repeating my question from duped bug 929988:
> 
> Why do we need the "> .anonymous-div" in those selectors?
> Wouldn't this be sufficient:
> 
> input:disabled, textarea:disabled {
>   cursor: default;
> }

I honestly can't remember if there is a reason why the rule is using "> .anonymous-div".

Comment 8

[Mats Palmgren (inactive)]

Attachment: fix

OK, this seems to work as expected for
  data:text/html,<input disabled style="cursor: not-allowed">
and
  data:text/html,<input disabled>

Comment 9

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 826066 [details] [diff] [review]
fix

r=me

Comment 10

[Mats Palmgren (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2721917a1b53

Comment 11

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/2721917a1b53

Comment 12

[ganesh_sahai1]

Verified on x86_64 Win7 using the latest nightly build and found it is fixed.

Comment 13

[daniel]

This bug still exists on Firefox 26.0.  Can be recreated by initial comment's JS Fiddle.

Comment 14

[temp]

This definitely isn't resolved in Firefox 26.0. I noticed it when trying to use Amazon's login page. They use that feature to disable their password field if you select New Account. I confirmed this works fine in Chrome 31 and IE11 but not FF26.

Comment 15

[Boris Zbarsky [:bzbarsky]]

> This definitely isn't resolved in Firefox 26.0.

Sure.  That's why the target milestone field says "Firefox 28", which is where it's resolved.