Closed
Bug 1377227
Opened 7 years ago
Closed 7 years ago
Bookmark and Page Action icons in the location bar are too dark
Categories
(Firefox :: Theme, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox54 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | verified |
People
(Reporter: Virtual, Assigned: Gijs)
References
(Blocks 1 open bug)
Details
(Keywords: nightly-community, ux-consistency, Whiteboard: [photon-structure] [fixed by patch from bug #1377165])
Attachments
(2 files)
Standardize buttons icons colors styles in address bar.
"☆" and "•••" buttons are too eye catchy now, as they are too black.
So either make them less black
or make other icons also that black,
so all buttons icons will have consistent color style
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
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
Whiteboard: [photon][triage] → [photon] [triage]
Assignee | ||
Updated•7 years ago
|
Whiteboard: [photon] [triage] → [photon-visual] [triage]
Assignee | ||
Comment 2•7 years ago
|
||
Actually, the structure team should probably own this ourselves given we're the ones adding most of these items, unless the visual team particularly wants to do this work. Dão, can you clarify how you're doing the colours in the navbar? It seems the design says to use #0c0c0d (#f9f9fa on inverted themes) on all of these items, but with 80% opacity in the navbar and 60% in the location bar. But the code I'm finding uses opaque colours that don't seem to match this ( https://dxr.mozilla.org/mozilla-central/source/browser/themes/shared/toolbarbutton-icons.inc.css#2-4 ). Is the design out of date, does it not match the live mocks, or am I just misunderstanding how it works (is there opacity applied somewhere else that I'm not seeing?) - and most importantly, how do we integrate the page action icons into however this system works?
Comment 3•7 years ago
|
||
The inversion logic is based on the toolbar's text color, which can be different from the location bar's text color. This means that you should not use --toolbarbutton-icon-fill here and you shouldn't hardcode #0c0c0d either. Since the difference between #0c0c0d (gray with blue shift) and #0c0c0c (real gray) is negligible, you should use currentColor with reduced opacity or fill-opacity.
Flags: needinfo?(dao+bmo)
Updated•7 years ago
|
Component: Toolbars and Customization → Theme
Updated•7 years ago
|
Summary: Standardize buttons icons colors styles in address bar → Bookmark and Page Action icons in the location bar are too dark
Updated•7 years ago
|
OS: Windows 7 → All
Hardware: x86_64 → All
Updated•7 years ago
|
Whiteboard: [photon-structure] → [photon-structure] [triage]
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → RESOLVED
Iteration: --- → 56.2 - Jul 10
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Whiteboard: [photon-structure] [triage] → [photon-structure][fixed by bug 1377165]
Updated•7 years ago
|
Flags: qe-verify+
Priority: -- → P1
QA Contact: gwimberly
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Comment 4•7 years ago
|
||
I'm marking this bug as VERIFIED, as patch from bug #1377165 fixed also this issue, starting from Mozilla Firefox Nightly 56.0a1 (2017-07-04).
Thanks.
Status: RESOLVED → VERIFIED
Depends on: 1377165
Whiteboard: [photon-structure][fixed by bug 1377165] → [photon-structure] [fixed by patch from bug #1377165]
Updated•7 years ago
|
Flags: qe-verify+
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
QA Contact: gwimberly → Virtual
Updated•7 years ago
|
Target Milestone: --- → Firefox 56
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
status-firefox54:
--- → unaffected
status-firefox55:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Version: Trunk → 56 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•