Closed
Bug 344588
Opened 19 years ago
Closed 17 years ago
[newprefs] Certificates preference associated label not spoken by WindowEyes
Categories
(Firefox :: Disability Access, defect)
Tracking
()
RESOLVED
FIXED
Firefox 3 beta2
People
(Reporter: pilgrim, Assigned: MarcoZ)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
asaf
:
review+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
Requires WindowEyes 5.5K and a Firefox build with the newly reorganized preferences window (bug 340677).
Step to reproduce:
1. Tools / Options / Advanced / Encryption
2. Tab to radio button in Certificates group box
Results:
WindowEyes speaks "Select one automatically"
Expected results:
WindowEyes should also speak the associated label, i.e. "When a web site requires a certificate: Select one automatically."
Reporter | ||
Updated•19 years ago
|
Target Milestone: --- → Firefox 2 beta2
Reporter | ||
Updated•19 years ago
|
Assignee: nobody → jwalden+bmo
Reporter | ||
Comment 1•18 years ago
|
||
Problem also exists in Firefox 1.5. Removing "regression" keyword.
Keywords: regression
Reporter | ||
Comment 2•18 years ago
|
||
*** Bug 347235 has been marked as a duplicate of this bug. ***
Updated•18 years ago
|
Assignee | ||
Updated•17 years ago
|
Assignee: jwalden+bmo → marco.zehe
Assignee | ||
Updated•17 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•17 years ago
|
||
This patch addresses the issue that currently no label is spoken when landing on the radio buttons. This does, however, not pick up the fact that we're in the Certificates groupbox. Only when tabbing onwards from the radio button group to the buttons, do screen readers pick this up. There does not appear to be a way of also including the caption for the groupbox in the aaa:labelledby attribute. I tried, and it fails.
Attachment #284629 -
Flags: review?(mano)
Assignee | ||
Comment 4•17 years ago
|
||
Ginn, do you have an idea why the caption is not included in the accessible name for the radio group, if I also give the caption an ID, and put that ID alongside the ID for the description in the aaa:labelledby attribute?
Assignee | ||
Comment 5•17 years ago
|
||
This patch shows the way I really would like it to come out in the end. However for this to work, changes in code may be required for the ID of the XUL:caption to be linked in properly.
(In reply to comment #4)
> Ginn, do you have an idea why the caption is not included in the accessible
> name for the radio group, if I also give the caption an ID, and put that ID
> alongside the ID for the description in the aaa:labelledby attribute?
>
I'm not sure.
Macro,
that's a bug. sigh.
Technically caption is not a label.
And we don't create accessible object for caption, we create an accessible object for the label inside it.
Check accessible/src/base/nsAccessible.cpp
nsAccessible::AppendFlatStringFromContentNode
Line 1587-1600
BTW: there's a typo in your patch
control="certselection" should be control="certSelection"
Comment on attachment 284629 [details] [diff] [review]
patch that links the description and the radio group together so the radio group appears properly labelled.
certselection should be
certSelection
It's case-sensitive.
Attachment #284629 -
Flags: review?(mano) → review-
Assignee | ||
Comment 9•17 years ago
|
||
(In reply to comment #7)
> Technically caption is not a label.
> And we don't create accessible object for caption, we create an accessible
> object for the label inside it.
OK, so the "only" part is: How do I reference that label from another control to have it describe/label something? :-)
> BTW: there's a typo in your patch
> control="certselection" should be control="certSelection"
Thanks for the pointer! That's a part I actually didn't touch, so the typo has been there all along. I've corrected it and will put up a new patch soon.
So the remaining question is: Do I have a means to reference that label, or is this currently impossible?
Comment 10•17 years ago
|
||
If you want to have the <caption> as part of the accessible name for radio group.
You can file a bug and assign it to Evan Yan or me.
Thanks!
Assignee | ||
Comment 11•17 years ago
|
||
Addresses typo in control attribute.
I will add a bug for the caption that cannot be referenced by aaa:labelledby, as also shown in this patch.
Attachment #284629 -
Attachment is obsolete: true
Attachment #284647 -
Attachment is obsolete: true
Attachment #286537 -
Flags: review?(ginn.chen)
Attachment #286537 -
Flags: review?(ginn.chen) → review?(mano)
Comment 12•17 years ago
|
||
Comment on attachment 286537 [details] [diff] [review]
Updated patch
r=mano
Attachment #286537 -
Flags: review?(mano) → review+
Attachment #286537 -
Flags: approval1.9?
Updated•17 years ago
|
Attachment #286537 -
Flags: approval1.9? → approval1.9+
Updated•17 years ago
|
Keywords: checkin-needed
Comment 13•17 years ago
|
||
Checking in browser/components/preferences/advanced.xul;
/cvsroot/mozilla/browser/components/preferences/advanced.xul,v <-- advanced.xul
new revision: 1.42; previous revision: 1.41
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: Firefox 2 beta2 → Firefox 3 M10
Depends on: 403724
You need to log in
before you can comment on or make changes to this bug.
Description
•