Closed Bug 1751282 Opened 3 years ago Closed 2 years ago

Creating a new address book or mailing list is not keyboard accessible in the new AB

Categories

(Thunderbird :: Address Book, defect, P1)

Thunderbird 98

Tracking

(thunderbird_esr102? fixed, thunderbird104 fixed)

RESOLVED FIXED
105 Branch
Tracking Status
thunderbird_esr102 ? fixed
thunderbird104 --- fixed

People

(Reporter: thomas8, Assigned: micah)

References

(Blocks 1 open bug)

Details

(Keywords: access, regression)

Attachments

(2 files)

98.0a1 (2022-01-20) (64-bit), Win10

No keyboard way of creating a new custom address book or mailing list in the NEW address book.

Expected:

  • Menu entries: File > New > Address Book (and Add CardDav Address Book / Add LDAP Address book)
  • Menu entry: File > New > Mailing List
  • Respective context menu entries somewhere, e.g.: Directory pane context menu.
  • Maybe the AB toolbar could also have a single tab focus stop on New Address Book button, then cursor keys to navigate adjacent buttons
Severity: -- → S3
Priority: -- → P3
Version: unspecified → Thunderbird 98
No longer blocks: tb-new-addrbook

See https://bugzilla.mozilla.org/show_bug.cgi?id=1705276#c6.

There is also no way to modify address books or mailing lists via keyboard.

(In reply to Nicolai Kasper from comment #1)

See https://bugzilla.mozilla.org/show_bug.cgi?id=1705276#c6.
There is also no way to modify address books or mailing lists via keyboard.

What do you mean by "modify address books"?

Given that the string freeze is approaching next tuesday, if we have menu entries like you suggested they will need strings. So you might want to push a patch that just adds these menu strings in case we need them, similarly regarding bug 1751282.

Otherwise, we will need to put a tabindex="0" on this element https://searchfox.org/comm-central/rev/2cf3a09a0127fe73e932c5ee793986f3bed2ccd2/mail/components/addrbook/content/aboutAddressBook.xhtml#41 and all the other toolbar buttons to have some way to add an addressbook, similarly regarding bug 1751282.

Mass-adding regression keyword to applicable AB bugs.

#20220530ABRegressionKeyword01

Keywords: regression

Alex, can we avoid making keyboard/screen reader depending users suffer for a full year with AB and mailing list creation inaccessible? Imo especially being unable to create a mailing list via keyboard looks like a pretty painful limitation.

  • Either restore some of the main menu regressions mentioned in comment 0 (I guess we could always show those entries as long as they open up AB tab), which is part of bug 1771605.
  • Or use Henry's quick fix of comment 3, tabindex="0" on the four AB toolbar buttons to make them accessible? This won't disturb regular navigation as we're focusing the search box when AB tab is opened, and the buttons are all before the search box in the tab sequence. Then later on at a time of our convenience we can reduce the four stops to one per toolbar in bug 1776192. This might be a fast and elegant solution, also very easy to undo. What do you think?
Flags: needinfo?(alessandro)

Yup, let's do that.
Micah, can you take care of this by following Henry's recommendation?

Otherwise, we will need to put a tabindex="0" on this element https://searchfox.org/comm-central/rev/2cf3a09a0127fe73e932c5ee793986f3bed2ccd2/mail/components/addrbook/content/aboutAddressBook.xhtml#41 and all the other toolbar buttons to have some way to add an addressbook, similarly regarding bug 1751282.

Make sure to test that tabbing will make possible to focus those buttons and trigger the creation dialogs when pressing Enter.

making this a P1 since it should happen pretty quickly

Assignee: nobody → micah
Flags: needinfo?(alessandro)
Priority: P3 → P1

It seems that macOS completely ignores tabindex for xul:toolbarbutton.
I also tried adding -moz-user-focus: normal but no luck, the focus is completely ignored and skipped.

Is this something xul toolbarbuttons don't support at all?
Kindly pinging Magnus to know if he has any clue, or if he knows someone we can ask from m-c.

Flags: needinfo?(mkmelin+mozilla)
Attached image focus.png (deleted) —

(In reply to Alessandro Castellani [:aleca] from comment #8)

It seems that macOS completely ignores tabindex for xul:toolbarbutton.
I also tried adding -moz-user-focus: normal but no luck, the focus is completely ignored and skipped.

Is this something xul toolbarbuttons don't support at all?
Kindly pinging Magnus to know if he has any clue, or if he knows someone we can ask from m-c.

This works for me on Mac and also the other platforms. Where is an accessibility setting on macOS to enable the focus ring on all elements.

On all platforms the focus ring is cut on the first button because of the outline-offset. Also does nothing happen when I press ENTER, SPACE or Cursor-DOWN.

(In reply to Richard Marti (:Paenglab) from comment #9)

On all platforms the focus ring is cut on the first button because of the outline-offset. Also does nothing happen when I press ENTER, SPACE or Cursor-DOWN.

XUL toolbarbutton-menubutton-button uses Alt+DownArrow for opening the menu. It is not very good, but it is the same for xul:menu.

Flags: needinfo?(mkmelin+mozilla)
Target Milestone: --- → 105 Branch

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/4fd17e6552d8
Make toolbar buttons keyboard accessible in Address Book. r=aleca

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Pushed by geoff@darktrojan.net: https://hg.mozilla.org/comm-central/rev/3eff6cdf41b6 follow-up - Fix a broken test. rs=bustage-fix

Comment on attachment 9288671 [details]
Bug 1751282 - Make toolbar buttons keyboard accessible in Address Book. r=aleca

[Approval Request Comment]
Regression caused by (bug #): bug 1705276
User impact if declined: Inability to access the creation buttons with keyboard navigation
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): it should be low since this only adds the tabindex attribute to those buttons to make them available in the regular Tab roving.

IMPORTANT! The follow up test fix patch should be uplifted alongside this.

Attachment #9288671 - Flags: approval-comm-esr102?
Attachment #9288671 - Flags: approval-comm-beta?

Comment on attachment 9288671 [details]
Bug 1751282 - Make toolbar buttons keyboard accessible in Address Book. r=aleca

[Triage Comment]
Approved for beta

Attachment #9288671 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9288671 [details]
Bug 1751282 - Make toolbar buttons keyboard accessible in Address Book. r=aleca

[Triage Comment]
Approved for esr102 (optimistically)

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

Attachment

General

Created:
Updated:
Size: