Closed Bug 479576 Opened 13 years ago Closed 13 years ago

Port Thunderbird Bug 449560 – Accessibility: Give header email labels (From/To/CC) ARIA label properties in message reading window/pane

Categories

(SeaMonkey :: MailNews: Message Display, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: MarcoZ, Assigned: MarcoZ)

References

Details

(Keywords: access)

Attachments

(1 file)

Lim Yuen Hoe of the univeristy of Singapore created this patch for Thunderbird, and it just landed there. Since the SeaMonkey mail reader is very similar, I ported (or rather, copy and pasted) the patch and request that SeaMonkey also include this bugfix, which will increase accessibility.
Flags: wanted-seamonkey2?
Attached patch Ported patchSplinter Review
Basically the same patch as in Thunderbird, with one modification: The header type label already contains a colon and a space, so don't add them here since that would add extraneous stuff that's not needed.
Attachment #363464 - Flags: review?
Attachment #363464 - Flags: review? → review?(iann_bugzilla)
Depends on: 479579
Attachment #363464 - Flags: review?(iann_bugzilla) → review+
Comment on attachment 363464 [details] [diff] [review]
Ported patch

It might be worth just adding in the comment why we don't have to put a colon and space in.
(In reply to comment #2)
> It might be worth just adding in the comment why we don't have to put a colon
> and space in.

Done, thanks!
Pushed:
http://hg.mozilla.org/comm-central/rev/b65ce77398b0
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
I think for SeaMonkey/MailNews you need a sr
Assignee: nobody → marco.zehe
Oops, this was in suite/mailnews, not in the root mailnews folder, so I thought this didn't need SR. The module owner's page didn't specify it either.
moa+=me after-the-fact, but please ask for sr next time. ;-)
Thanks, will do! And sorry about that!
Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.9.1b3pre) Gecko/20090301 SeaMonkey/2.0b1pre.
Status: RESOLVED → VERIFIED
Flags: wanted-seamonkey2?
You need to log in before you can comment on or make changes to this bug.