Closed
Bug 492518
Opened 15 years ago
Closed 15 years ago
xul:slider accessible of xul:scale is accessible illegally
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: access, verified1.9.1)
Attachments
(1 file)
|
4.58 KB,
patch
|
MarcoZ
:
review+
davidb
:
review+
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
it's happen with xul:scale accessible which contains xul:slider element, we should check this case. This prevents AT to grab name from slider accessible of video element (see bug 490287).
|
Assignee | |
Comment 1•15 years ago
|
||
changing summary to reflect problem better
Summary: focus event can be fired for non accessible anonymous content → xul:slider accessible of xul:scale is accessible illegally
|
|
Assignee | |
Comment 2•15 years ago
|
||
Ok, we create because xul:scale allows accessibles in anonymous content because of xul:thumb (btw, see bug 492513) and generic accessible for xul:slider is created because it is focusable.
|
|
Assignee | |
Comment 3•15 years ago
|
||
Attachment #377104 -
Flags: review?(marco.zehe)
Attachment #377104 -
Flags: review?(david.bolter)
|
|
Assignee | |
Comment 4•15 years ago
|
||
disable accessible in anonymous content, including thumb accessibles - it sounds slider accessible lives successfully without thumb accessibles (bug 492513).
|
||
Updated•15 years ago
|
Attachment #377104 -
Flags: review?(david.bolter) → review+
|
|
||
Comment 5•15 years ago
|
||
Comment on attachment 377104 [details] [diff] [review] patch r=me (unless the vendors disagree re: email) for the non-test part. I'm still not quite sure how the test covers the bug.
|
|
||
Comment 6•15 years ago
|
||
OK IRC chat made me happy for the test part :)
|
||
Comment 7•15 years ago
|
||
Comment on attachment 377104 [details] [diff] [review] patch r=me as well.
Attachment #377104 -
Flags: review?(marco.zehe) → review+
|
|
Assignee | |
Comment 8•15 years ago
|
||
checked in http://hg.mozilla.org/mozilla-central/rev/768064b2ee15 without AT approval because any way it's pure crossplatform fix which doesn't affect on AT and it makes HTML 5 audio/video more accessible. Let's move slider accessibility discussions to bug 492513.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•15 years ago
|
Flags: in-testsuite+
|
|
Assignee | |
Updated•15 years ago
|
Attachment #377104 -
Flags: approval1.9.1?
|
|
Assignee | |
Comment 9•15 years ago
|
||
Comment on attachment 377104 [details] [diff] [review] patch needed for HTML5 audio/video accessibility
|
||
Comment 10•15 years ago
|
||
Comment on attachment 377104 [details] [diff] [review] patch a191=beltzner
Attachment #377104 -
Flags: approval1.9.1? → approval1.9.1+
|
|
Assignee | |
Comment 11•15 years ago
|
||
checked in on 1.9.1 - http://hg.mozilla.org/releases/mozilla-1.9.1/rev/63f9e39a84ae
Keywords: access,
fixed1.9.1
|
|
||
Comment 12•15 years ago
|
||
Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090517 Minefield/3.6a1pre (.NET CLR 3.5.30729)
Status: RESOLVED → VERIFIED
|
|
||
Comment 13•15 years ago
|
||
Verified fixed on 1.9.1 in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b5pre) Gecko/20090517 Shiretoko/3.5b5pre (.NET CLR 3.5.30729)
Keywords: fixed1.9.1 → verified1.9.1
You need to log in
before you can comment on or make changes to this bug.
Description
•