Closed Bug 1698761 Opened 4 years ago Closed 3 years ago

[respect-system-appearance] Respect macOS system theme in native-styled menu arrows

Categories

(Core :: Widget: Cocoa, task, P2)

task
Points:
3

Tracking

()

RESOLVED DUPLICATE of bug 1710459

People

(Reporter: mbalfanz, Unassigned)

References

Details

(Whiteboard: [mac:darkmode][proton-toolbar])

Attachments

(4 files, 1 obsolete file)

Attached image Screenshot 2021-03-16 at 11.20.18.png (deleted) —

With the patch from bug 1697331, and widget.macos.respect-system-appearance set to true, nested items in the bookmarks toolbar looks like in the attached screenshot in macOS Dark Mode.

The problems I see are:

  • The background is too bright. Compare this to Safari in the next screenshot
  • The arrow appears dark when it should be light
Attached image safari.png (obsolete) (deleted) —

I realize that the "background being too bright" is invalid. I'll remove it from comment 0. The remaining problem is the black arrow.

Attachment #9209408 - Attachment is obsolete: true
Severity: -- → S2
Priority: -- → P2
Whiteboard: [mac:darkmode] → [mac:darkmode][proton-toolbar]
Attached image With patch from bug 1703051 applied (deleted) —

Looks like this will get fixed as part of bug 1703051.

Attached image image.png (deleted) —

(In reply to Mike Conley (:mconley) (:⚙️) (Catching up on needinfos) from comment #3)

Looks like this will get fixed as part of bug 1703051.

Sort of. This will still be an issue in any other non-native but native-styled nested menus. The outstanding issue is the black arrow, which should be replaced with something that respects the system theme. I'll update the bug accordingly.

Summary: Make bookmarks toolbar look good in dark mode → [respect-system-appearance] Respect macOS system theme in native-styled menu arrows
Points: --- → 3
Assignee: nobody → bigiri
Status: NEW → ASSIGNED

Where can I find "non-native but native-styled nested menus" that still show this black arrow?

The only similar arrows I could find was this: https://searchfox.org/mozilla-central/search?q=item-arrow&path=&case=false&regexp=false

Which is only used in Devtools in the Style Editor.

Flags: needinfo?(htwyford)

There's a couple in the dark mode Library window toolbarbuttons. Screenshot attached. This bug probably doesn't deserve to be a P2/S2, since these arrows are fairly rare now and a fix would be difficult. Afaik, these arrows are included in CoreUI under the names MenuSubmenu and MenuSubmenuLeft. We reference them here and they're drawn here. Since the icons aren't dark-mode aware, we'd have to refactor our non-native menu code to draw these arrows in a more modern way.

Markus, am I missing something here?

Flags: needinfo?(htwyford) → needinfo?(mstange.moz)

I think we could use [NSImage imageNamed:@"chevron.right"] for this, or imageNamed:NSImageNameRightFacingTriangleTemplate. Those special NSImages use dark-mode aware drawing. But we'd need to test what chevron.right draws on older macOS versions.
But given that these menus are now super rare in Firefox, and that bug 1710459 will at some point take care of them, I'm not sure if it's worth spending time on this bug.

Flags: needinfo?(mstange.moz)
Assignee: bigiri → nobody
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: