Closed Bug 729845 Opened 9 years ago Closed 9 years ago
"disabled" attribute should not work on non-form elements
User Agent: Mozilla/5.0 (Ubuntu; X11; Linux i686; rv:9.0.1) Gecko/20100101 Firefox/9.0.1 Build ID: 20111228084953 Steps to reproduce: The attached page contains 2 divs with tabIndex=1 and one of them is disabled. Actual results: If I tab to the disabled element outline is not drawn which means "disabled" attribute is working for non-form control elements. Expected results: "disabled" should not work for non-form control elements.
Please also refer webkit bugs:- https://bugs.webkit.org/show_bug.cgi?id=22202
Date: Wed, 22 Feb 2012 10:08:27 -0500 From: Boris Zbarsky <email@example.com> ... Looks like just a bug to me. Please file in the Core product? -Boris
Component: Untriaged → General
Product: Firefox → Core
QA Contact: untriaged → general
Mounir, shouldn't IsDisabled() on nsGenericHTMLElement just return false?
Component: General → DOM
QA Contact: general → general
Indeed, with bug 557087, I did introduce |IsDisabled()| but it seems that this bug didn't cause the regression: https://hg.mozilla.org/mozilla-central/rev/b15d9c989201#l1.157 So I guess the described issue must be quit old. Here is a patch that might fix it. I will write some tests as soon as I will have a try run done.
Assignee: nobody → mounir
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #603999 - Flags: review?(bzbarsky)
OS: Linux → All
Hardware: x86 → All
Whiteboard: [needs review]
Version: 9 Branch → Trunk
I think this pass should pass our test suite. It's lovely to see that we had tests checking that <div tabindex=1 disabled> was actually not focusable... :)
Whiteboard: [needs review] → [needs review][passes try]
Comment on attachment 604033 [details] [diff] [review] Patch with tests r=me
Attachment #604033 - Flags: review?(bzbarsky) → review+
Whiteboard: [needs review][passes try]
Target Milestone: --- → mozilla13
Antaryami, thank you for reporting that issue. That was a very valuable feedback! That bug should be fixed in Firefox 13. The fix will be available soon in our Nightly builds.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.