1738942 - Accessibility regression: wrong item count in new Address Book contact list

Status: RESOLVED FIXED

(Thunderbird :: Address Book, defect)

Description

[Valentin Villenave]

This is a rather minor bug, loosely related to bug 1738288
(and first investigated by Joanmarie Diggs.
Like that one, it’s only exposed through TB’s at-spi output
so a screen reader such as Orca is required to reproduce it.

• Open Orca’s preferences (with orca -s).
• In the Speech tab, check the “Speak child position” box.
• Apply (this will relaunch Orca).
• Open TB’s new Address Book with at least three contacts in there.
• Access the search entry field and hit Tab twice;
• then hit the Down arrow to move onto the next contact.

Instead of reading “item 2 of n”, Orca’s now reading “item 1 of n”.
It appears to be because TB’s “exposing the wrong value for
posinset and setsize” (Joanmarie’s words, not mine).

Comment 1

[Thomas D. (:thomas8)]

Mass-adding regression keyword to applicable AB bugs.

#20220530ABRegressionKeyword01

Comment 2

[Henry Wilkes (they/them) [:henry-x]]

Hey, I just mentioned this in bug 1752532 comment 14. This will be addressed in that bug, but I might jump onto this one to fix this specific issue for 102.

Comment 3

[Henry Wilkes (they/them) [:henry-x]]

Attachment: Bug 1738942 - Use the correct number for aria-posinset in treeview-listbox. r=darktrojan

Depends on D149421

Comment 4

[Henry Wilkes (they/them) [:henry-x]]

Unblocking from bug 1752532. I'm fix this for the current tree-view-listbox for 102.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/d4048bb9de2c
Use the correct number for aria-posinset in treeview-listbox. r=darktrojan

Comment 6

[Henry Wilkes (they/them) [:henry-x]]

Comment on attachment 9281465 [details]
Bug 1738942 - Use the correct number for aria-posinset in treeview-listbox. r=darktrojan

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Screen reader announces the wrong number for list items.
Testing completed (on c-c, etc.): No
Risk to taking this patch (and alternatives if risky): Low. Small fix.

Comment 7

[Wayne Mery (:wsmwk)]

Comment on attachment 9281465 [details]
Bug 1738942 - Use the correct number for aria-posinset in treeview-listbox. r=darktrojan

[Triage Comment]
Approved for esr102

Comment 8

[Rob Lemley [:rjl]]

Thunderbird 102.0:
https://hg.mozilla.org/releases/comm-esr102/rev/4c5635269db9