Open Bug 1703008 Opened 4 years ago Updated 3 years ago

Some folder tree and icon colours are wrong in the Bookmarks Panel with Dark Theme enabled

Categories

(Firefox :: Bookmarks & History, task, P3)

task

Tracking

()

People

(Reporter: mconley, Unassigned, NeedInfo)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-door-hangers] [priority:2c])

Attachments

(1 file)

Specifically, the icons for the expanders, the menulist dropdown icon, the new folder icon, and the tree twisties in the folder selector.

Blocks: proton-door-hangers
No longer blocks: proton
Whiteboard: [proton-door-hangers]
Priority: -- → P2
Whiteboard: [proton-door-hangers] → [proton-door-hangers][priority:2a]
Whiteboard: [proton-door-hangers][priority:2a] → [proton-door-hangers] [priority:2a]
See Also: → 1707875
Assignee: nobody → mconley

So I dug into this a bit, and it looks like since this bug was originally filed, the colours have been corrected for dark mode for:

  • The expander icons
  • The menulist icon
  • The new folder button icon

However, it looks like the folder menulist still needs to be restyled. According to the panels MR1 experience review, the hover and selected states are not exactly correct. I'm going to consult with KatieC on what is expected here.

Status: NEW → ASSIGNED
Whiteboard: [proton-door-hangers] [priority:2a] → [proton-door-hangers] [priority:2c]

The priority on this bug has gone down, so I'm unassigning myself from it to work on higher priority things.

Assignee: mconley → nobody
Status: ASSIGNED → NEW
Severity: -- → S3
Priority: P2 → P3

mconley, can you check if this bug is still actionable? If so, what changes are required exactly for the folder menulist?

Flags: needinfo?(mconley)
Attached image image.png

Yeah, I think the issue is still here - it has to do with the selected item colour. I believe UX wanted that selected item colour to match similar selections with Dark Mode (--button-primary-hover-bgcolor).

Flags: needinfo?(mconley)
Flags: needinfo?(itiel_yn8)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: