Closed Bug 1856396 Opened 2 years ago Closed 2 years ago

Add aria labels to message cells in table view

Categories

(Thunderbird :: Disability Access, defect, P2)

Thunderbird 115

Tracking

(thunderbird_esr115 fixed, thunderbird119 wontfix)

RESOLVED FIXED
120 Branch
Tracking Status
thunderbird_esr115 --- fixed
thunderbird119 --- wontfix

People

(Reporter: elijmitchell, Assigned: elijmitchell)

References

(Blocks 2 open bugs)

Details

(Keywords: access)

Attachments

(1 file)

Add aria-labels to individual cells to be used by screen readers.

Attachment #9356203 - Attachment description: WIP: Bug 1856396 - Add aria labels to message table cells. r=rjl,#thunderbird-front-end-reviewers → Bug 1856396 - Add aria labels to message table cells. r=rjl,#thunderbird-front-end-reviewers
Blocks: 1856407
Attachment #9356203 - Attachment description: Bug 1856396 - Add aria labels to message table cells. r=rjl,#thunderbird-front-end-reviewers → Bug 1856396 - Add aria labels to message table cells. r=freaktechnik
Target Milestone: --- → 120 Branch

Pushed by elizabeth@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/edf24587c70d
Add aria labels to message table cells. r=freaktechnik

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED

Comment on attachment 9356203 [details]
Bug 1856396 - Add aria labels to message table cells. r=freaktechnik

[Triage Comment]
Approved for beta

Attachment #9356203 - Flags: approval-comm-beta+

Comment on attachment 9356203 [details]
Bug 1856396 - Add aria labels to message table cells. r=freaktechnik

[Triage Comment]
l10n - so must ride the trains to beta, don't uplift

Attachment #9356203 - Flags: approval-comm-beta+ → approval-comm-beta-
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

(In reply to Daniel Darnell [:dandarnell] from comment #6)

Backed out from Thunderbird 119.0b3:
https://hg.mozilla.org/releases/comm-beta/rev/3651cfa9a963

Hi. This is merged into 120. So this being reopened doesn't seem like the right status, correct?

In terms of this going into 115 or beta, I don't think there is much that can be done due to our translation workflows. But this is already in daily.

Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
See Also: → 1858825

Comment on attachment 9356203 [details]
Bug 1856396 - Add aria labels to message table cells. r=freaktechnik

[Triage Comment]
Approved for beta to go with bug 1843486

Attachment #9356203 - Flags: approval-comm-beta- → approval-comm-beta+
Regressions: 1861204

This is tagged as high severity.
OK to uplift to 115, despite the regression?

Flags: needinfo?(rob)
Flags: needinfo?(elizabeth)

Looks like most locales have these translated, so I will repin l10n-changesets.json when this goes to comm-esr115.

Flags: needinfo?(rob)

(In reply to Wayne Mery (:wsmwk) from comment #10)

This is tagged as high severity.
OK to uplift to 115, despite the regression?

Sorry, Wayne, for the delay. The only delay on my end was with translations. So if Rob is okay with this, that is fine with me.

More related changes will be coming and we can make sure they get uplifted to comm-esr115 as well. Thank you!

Flags: needinfo?(elizabeth)

Comment on attachment 9356203 [details]
Bug 1856396 - Add aria labels to message table cells. r=freaktechnik

[Approval Request Comment]
Raising the request, per previous comments

Attachment #9356203 - Flags: approval-comm-esr115?

Comment on attachment 9356203 [details]
Bug 1856396 - Add aria labels to message table cells. r=freaktechnik

[Triage Comment]
Approved for esr115

Attachment #9356203 - Flags: approval-comm-esr115? → approval-comm-esr115+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: