Closed Bug 351946 Opened 19 years ago Closed 19 years ago

Feeds preferences, visual reader choice association must be explicit for screen readers

Categories

(Firefox :: Settings UI, defect)

Product:
Firefox
Component:
Settings UI
Version:
2.0 Branch
Platform:
x86
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 2

People

(Reporter: aaronlev, Assigned: aaronlev)

References

Details

(Keywords: access, fixed1.8.1)

Attachments

(1 file, 1 obsolete file)

Correct patch
1.84 KB, patch
asaf
: review+
mtschrep
: approval1.8.1+
Details | Diff | Splinter Review
Under Tools -> Options -> Feeds there is a radio button option, with an associated listbox. - When screen reading the radio button option for "Subscribe to feed using:" it should say something like "Subscribe to feed using: Live bookmarks". - Also, the listbox's label should be "Subscribe to feed using:" so that when focusing it a screen reader user knows what it's for. This fix requires the AppendFlatStringFromSubtree fix in bug 347219.
Version: Trunk → 2.0 Branch
Attached patch Use labelledby (obsolete) — — Splinter Review
Attachment #237495 - Flags: review?(bugs.mano)
Comment on attachment 237495 [details] [diff] [review] Use labelledby looks like you've attached the wrong patch.
Attachment #237495 - Flags: review?(bugs.mano)
Attached patch Correct patch — — Splinter Review
Attachment #237495 - Attachment is obsolete: true
Attachment #237555 - Flags: review?(bugs.mano)
Comment on attachment 237555 [details] [diff] [review] Correct patch r=mano
Attachment #237555 - Flags: review?(bugs.mano) → review+
Attachment #237555 - Flags: approval1.8.1?
Whiteboard: Low risk, subtle but significant in that it makes screen reader output sensible for that panel.
Whiteboard: Low risk, subtle but significant in that it makes screen reader output sensible for that panel. → No l10n impact, low risk, subtle but significant in that it makes screen reader output sensible for that panel.
Comment on attachment 237555 [details] [diff] [review] Correct patch a=schrep for accessibility fix for new feed preview.
Attachment #237555 - Flags: approval1.8.1? → approval1.8.1+
Keywords: fixed1.8.1
Whiteboard: No l10n impact, low risk, subtle but significant in that it makes screen reader output sensible for that panel. → [checkin needed]
Assignee: aaronleventhal → nobody
Assignee: nobody → aaronleventhal
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 2
Whiteboard: [checkin needed]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: