Closed Bug 1713429 Opened 2 years ago Closed 2 years ago

Inconsistent focus and hover effects of AB context menu button in contacts sidebar

Categories

(Thunderbird :: Theme, task)

Tracking

(thunderbird_esr78 wontfix)

RESOLVED FIXED
91 Branch
Tracking Status
thunderbird_esr78 --- wontfix

People

(Reporter: thomas8, Assigned: Paenglab)

Details

Attachments

(5 files, 2 obsolete files)

Hover and focus effects of AB context menu button in contacts sidebar are odd.

  • Focus: With Proton, dotted blue focus indication is no longer wanted - needs plain blue border.
  • Hover of focused button: suddenly showing the expected focus style (which is unexpected).
    See screenshots.
Attachment #9224117 - Attachment description: image.png → Screenshot 1: Proton-inconsistent focus style of AB context menu button

I am also seeing some "double" focus borders around interactive elements like buttons and dropdowns.
It looks a bit strange (flimsy? unpolished? or maybe just different?)
Maybe focus border should conflate with the element border, as seen on the search box and address row when they have focus?
Or could we remove the gap between the focus border and the element border?

For comparison, here's our identity selector dropdown (not structurally different from AB dropdown) which has single-border focus. I think that looks better, more calm, more focused :-)

Attachment #9224118 - Attachment description: Screenshot 2: Wrong hover style (this should be the focus style) → Screenshot 2: Unexpected style change when hovering focused button (something like this should be the focus style)
Summary: Wrong focus and hover effect of AB context menu button in contacts sidebar → Inconsistent focus and hover effects of AB context menu button in contacts sidebar

(In reply to Thomas D. (:thomas8) from comment #2)

Created attachment 9224120 [details]
Screenshot 3: Odd double focus border

I am also seeing some "double" focus borders around interactive elements like buttons and dropdowns.
It looks a bit strange (flimsy? unpolished? or maybe just different?)
Maybe focus border should conflate with the element border, as seen on the search box and address row when they have focus?
Or could we remove the gap between the focus border and the element border?

Welcome to Proton. :-)

I'll do our adaptions to the address sidebar.

Assignee: nobody → richard.marti
Status: NEW → ASSIGNED

This fixes the abContextMenuButton's focus ring.

The 1px gap is a Proton styling and I leave it now. FX uses 2px which is too much in the normal UI. In the prefs etc. we still use 2px.

The gap is also to better show the focus on checked checkmarks or radio buttons which have the same blue background and a focus on them would only grow the area which isn't so good visible.

Attachment #9224127 - Flags: review?(alessandro)

Made that the [default] button has always his border, also when another button has the focus. This is to mimic the behaviour of TB 68 which don't have the themeable dialogs. You can test it for example on a folder properties dialog.

Attachment #9224127 - Attachment is obsolete: true
Attachment #9224127 - Flags: review?(alessandro)
Attachment #9224164 - Flags: review?(alessandro)

Bah, found that Mac had its owns style for the focus, removed.

Attachment #9224164 - Attachment is obsolete: true
Attachment #9224164 - Flags: review?(alessandro)
Attachment #9224167 - Flags: review?(alessandro)
Comment on attachment 9224167 [details] [diff] [review]
1713429-abContextMenuButton-focus.patch

Review of attachment 9224167 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks for taking care of this.
Attachment #9224167 - Flags: review?(alessandro) → review+
Target Milestone: --- → 91 Branch

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/52cc1c44232a
Make the focus ring of the abContactsPanel's abContextMenuButton follow Proton. r=aleca

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.