Open
Bug 1378301
Opened 7 years ago
Updated 1 year ago
Bookmark star icon in context menu should be blue (not black) when filled in
Categories
(Firefox :: Menus, defect, P5)
Tracking
()
NEW
People
(Reporter: Virtual, Unassigned)
References
Details
(Keywords: nightly-community, ux-consistency)
Attachments
(1 file)
10.40 KB,
image/png
|
Details |
Bookmark star icon in context menu opened by mouse on tab content should be blue, not black, when filled in, to be consistent with other bookmarks icons.
Similar to bug #1377165, so I'm CC'ing :Gijs
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
Has Regression Range: --- → irrelevant
Has STR: --- → irrelevant
Comment 1•7 years ago
|
||
This has nothing to do with Photon - the context menu icon has been there for a lot longer (specifically bug 1000513 - Firefox 32, 3 years ago) and has never been blue, while the bookmarks star has always been blue.
Blocks: 1000513
Whiteboard: [photon] [triage]
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Comment 2•7 years ago
|
||
Yes. I know there is nothing for now in Photon mockups and specs about this, but it could be the right time to standardize and make this consistent.
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Comment 3•7 years ago
|
||
@ Jared Wein [:jaws] - I "needinfo'ing" you, so you will know about this issue.
We could finally standardize all bookmark star icons to be blue, not black, when filled in.
I think, it's right time to do that now, per Photon redesign.
Similar bugs about this issue are here: bug #1377165 and bug #1381158.
Flags: needinfo?(jaws)
Version: Trunk → 32 Branch
Comment 4•7 years ago
|
||
I agree that this is something we could do now (we couldn't do it in bug 1000513 because we didn't have context-fill). However we probably have too much Photon work right now to prioritize this.
Flags: needinfo?(jaws)
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
QA Contact: Virtual
Updated•7 years ago
|
Priority: -- → P5
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•