Default value of tabIndex IDL attribute for FIELDSET and OUTPUT
Reported by
taken....@gmail.com,
Aug 30 2016
|
|||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2843.0 Safari/537.36 Example URL: Steps to reproduce the problem: HTML Standard says that default value is 0 for elements that are focusable and −1. https://html.spec.whatwg.org/multipage/interaction.html#dom-tabindex Some non-focusable elements (listed below) return 0 for tabIndex. a without href input[type=hidden] input[disabled] select[disabled] textarea[disabled] button[disabled] filedset output What is the expected behavior? Non-focusable elements should return -1 for tabIndex What went wrong? Non-focusable elements return 0 Does it occur on multiple sites: Yes Is it a problem with a plugin? No Did this work before? No Does this work in other browsers? Yes Chrome version: 55.0.2843.0 Channel: canary OS Version: 10.0 Flash Version: Shockwave Flash 22.0 r0 It may be too late to change default valule of following elements as Firefox, Edge and Safari return 0 for years. a without href input[type=hidden] input[disabled] select[disabled] textarea[disabled] button[disabled] Firefox returns -1 for fieldset and output
,
Aug 30 2016
,
Aug 30 2016
Yeah, we shouldn't change the interoperable behavior. So this bug should handle only <fieldset> and <output>.
,
Aug 31 2016
If no one else is working on this, I'd like to contribute.
,
Sep 1 2016
I made a fix on this, not sure if I did it correctly. Could someone please take a look at it? https://codereview.chromium.org/2298133004/ Thanks
,
Sep 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8c5fd085e7108025af04004150ee65c09e93d404 commit 8c5fd085e7108025af04004150ee65c09e93d404 Author: lunalu <lunalu@chromium.org> Date: Fri Sep 02 03:36:44 2016 Changed the default value of tabIndex of <fieldset> and <output> to be -1 as they are non-focusable elements. BUG= 642267 Review-Url: https://codereview.chromium.org/2298133004 Cr-Commit-Position: refs/heads/master@{#416181} [add] https://crrev.com/8c5fd085e7108025af04004150ee65c09e93d404/third_party/WebKit/LayoutTests/fast/html/tabindex-nonfocusable.html [modify] https://crrev.com/8c5fd085e7108025af04004150ee65c09e93d404/third_party/WebKit/Source/core/html/HTMLFieldSetElement.cpp [modify] https://crrev.com/8c5fd085e7108025af04004150ee65c09e93d404/third_party/WebKit/Source/core/html/HTMLFieldSetElement.h [modify] https://crrev.com/8c5fd085e7108025af04004150ee65c09e93d404/third_party/WebKit/Source/core/html/HTMLFormControlElement.h [modify] https://crrev.com/8c5fd085e7108025af04004150ee65c09e93d404/third_party/WebKit/Source/core/html/HTMLOutputElement.cpp [modify] https://crrev.com/8c5fd085e7108025af04004150ee65c09e93d404/third_party/WebKit/Source/core/html/HTMLOutputElement.h
,
Sep 2 2016
,
Dec 9 2016
,
Jul 18 2017
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by taken....@gmail.com
, Aug 30 20162.0 KB
2.0 KB View Download