Closed
Bug 255881
Opened 20 years ago
Closed 20 years ago
REGRESSION: frames and iframes not tabbable
Categories
(Core :: DOM: UI Events & Focus Handling, defect)
Core
DOM: UI Events & Focus Handling
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
Details
(Keywords: access, regression)
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
neil
:
review+
bryner
:
superreview+
|
Details | Diff | Splinter Review |
Spun off from bug 254966, where mats noticed this.
Normally <frame> and <iframe> are in the tab order. It's critical to
accessibility that they are.
Test cases:
http://bugzilla.mozilla.org/attachment.cgi?id=153477&action=view
http://bugzilla.mozilla.org/attachment.cgi?id=153478&action=view
Assignee | ||
Comment 1•20 years ago
|
||
This is extremely important - 1.8a3 is going to be our accessibility testing
platform for a while.
Flags: blocking1.8a3?
Assignee | ||
Comment 2•20 years ago
|
||
Assignee | ||
Updated•20 years ago
|
Attachment #156345 -
Flags: superreview?(bryner)
Assignee | ||
Updated•20 years ago
|
Attachment #156345 -
Flags: review?(neil.parkwaycc.co.uk)
Assignee | ||
Updated•20 years ago
|
Attachment #156345 -
Flags: superreview?(bryner)
Attachment #156345 -
Flags: review?(neil.parkwaycc.co.uk)
Assignee | ||
Updated•20 years ago
|
Attachment #156345 -
Attachment is obsolete: true
Assignee | ||
Comment 3•20 years ago
|
||
Assignee | ||
Updated•20 years ago
|
Attachment #156349 -
Flags: superreview?(bryner)
Attachment #156349 -
Flags: review?(neil.parkwaycc.co.uk)
Comment 4•20 years ago
|
||
Comment on attachment 156349 [details] [diff] [review]
Better fix: make default tabIndex 0 for frame/iframe
Can't see anything wrong; tabindex="-1" still seems to work for instance.
Attachment #156349 -
Flags: review?(neil.parkwaycc.co.uk) → review+
Updated•20 years ago
|
Attachment #156349 -
Flags: superreview?(bryner) → superreview+
Assignee | ||
Comment 5•20 years ago
|
||
Comment on attachment 156349 [details] [diff] [review]
Better fix: make default tabIndex 0 for frame/iframe
Major regression.
Attachment #156349 -
Flags: approval1.8a3?
Updated•20 years ago
|
Flags: blocking1.8a3? → blocking1.8a3-
Updated•20 years ago
|
Flags: blocking1.8a4?
Assignee | ||
Comment 6•20 years ago
|
||
Checking in content/html/content/src/nsGenericHTMLElement.cpp;
/cvsroot/mozilla/content/html/content/src/nsGenericHTMLElement.cpp,v <--
nsGenericHTMLElement.cpp
new revision: 1.536; previous revision: 1.535
done
Checking in content/html/content/src/nsGenericHTMLElement.h;
/cvsroot/mozilla/content/html/content/src/nsGenericHTMLElement.h,v <--
nsGenericHTMLElement.h
new revision: 1.200; previous revision: 1.199
done
Assignee | ||
Comment 7•20 years ago
|
||
Checking in content/html/content/src/nsGenericHTMLElement.cpp;
/cvsroot/mozilla/content/html/content/src/nsGenericHTMLElement.cpp,v <--
nsGenericHTMLElement.cpp
new revision: 1.536; previous revision: 1.535
done
Checking in content/html/content/src/nsGenericHTMLElement.h;
/cvsroot/mozilla/content/html/content/src/nsGenericHTMLElement.h,v <--
nsGenericHTMLElement.h
new revision: 1.200; previous revision: 1.199
done
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Flags: blocking1.8a4?
Comment 8•20 years ago
|
||
Comment on attachment 156349 [details] [diff] [review]
Better fix: make default tabIndex 0 for frame/iframe
unsetting 1.8a3 approval request. we've shipped already.
Attachment #156349 -
Flags: approval1.8a3?
Updated•6 years ago
|
Component: Keyboard: Navigation → User events and focus handling
You need to log in
before you can comment on or make changes to this bug.
Description
•