Closed
Bug 489306
Opened 16 years ago
Closed 16 years ago
overflowed content doesn't expose child text accessibles
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
VERIFIED
FIXED
People
(Reporter: surkov, Assigned: surkov)
References
(Blocks 1 open bug)
Details
(Keywords: fixed1.9.1)
Attachments
(2 files)
|
(deleted),
patch
|
davidb
:
review+
MarcoZ
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
|
(deleted),
patch
|
Details | Diff | Splinter Review |
<div style="overflow: auto">long content</div> doesn't expose child text accessible. I suspect bug 483573 because there we introduced walking through frames if there is anonymous content. I have wip patch that fixes the problem (fix logic of a11y tree walker) and will post it once I'll polish it.
|
Assignee | |
Comment 1•16 years ago
|
||
short description: when I added querying to anonymous content creator then a11y tree walker founds scrollbar element in the case of overflowed div, scrollbar isn't accessible so it checks it first child , GetFirstChild is called again and then GetKids. GetKids sees it's XUL element so it nulls frame of current state before we put the current state on stack.
Attachment #373824 -
Flags: review?(david.bolter)
|
|
Assignee | |
Updated•16 years ago
|
Attachment #373824 -
Flags: review?(marco.zehe)
|
|
Assignee | |
Updated•16 years ago
|
Status: NEW → ASSIGNED
|
||
Comment 2•16 years ago
|
||
Comment on attachment 373824 [details] [diff] [review]
patch
>+ * DOM/frame subtree duiring searching of accessible children.
Nit: Typo "diuring" instead of "during".
>+ * Pop state from stack and makes it current.
Nit: Either "pops state...and makes..." or "pop state...and make it..." I'd prefer the latter (the one with pop and make).
Attachment #373824 -
Flags: review?(marco.zehe) → review+
|
||
Comment 3•16 years ago
|
||
Comment on attachment 373824 [details] [diff] [review]
patch
r=me
Attachment #373824 -
Flags: review?(david.bolter) → review+
|
|
Assignee | |
Comment 4•16 years ago
|
||
Attachment #373824 -
Flags: approval1.9.1?
|
|
Assignee | |
Comment 5•16 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
|
||
Comment 6•16 years ago
|
||
Comment on attachment 373824 [details] [diff] [review]
patch
a191=beltzner
Attachment #373824 -
Flags: approval1.9.1? → approval1.9.1+
|
|
Assignee | |
Comment 7•16 years ago
|
||
|
|
||
Comment 8•16 years ago
|
||
Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090503 Minefield/3.6a1pre (.NET CLR 3.5.30729)
Status: RESOLVED → VERIFIED
|
|
Assignee | |
Comment 9•16 years ago
|
||
checked in on mozilla1.9.1 http://hg.mozilla.org/releases/mozilla-1.9.1/rev/fe1a4c1e4fa0
Keywords: fixed1.9.1
You need to log in
before you can comment on or make changes to this bug.
Description
•