Closed Bug 947356 Opened 10 years ago Closed 10 years ago

On a dark/black toolbar, icons are too dark, need lighter color for more contrast (inverted icons?).

Categories

(Firefox :: Theme, defect)

x86_64
Linux
defect
Not set
normal
Points:
2

Tracking

()

VERIFIED FIXED
Firefox 33
Iteration:
33.2

People

(Reporter: Aleksej, Assigned: dao)

References

(Depends on 1 open bug, Blocks 2 open bugs)

Details

(Keywords: regression)

Attachments

(2 files)

2013-12-06-03-02-02-mozilla-central-firefox-28.0a1.en-US.linux-x86_64

The screenshots were made with a "Shiki" GTK theme.  The zoom control is text, so it is a separate issue (it works with Darklooks).
Which icons do you mean? The tabscrolling and tab list dropdown icons, or something else? In the screenshot, it seems all other icons are inverted...
Flags: needinfo?(deletesoftware+moz)
I think the "+", "−" and wrench icons are not inverted: they look wrong.
Flags: needinfo?(deletesoftware+moz)
I think this is basically unsolvable (for now) - no matter what icon color/inversion we use, there will be some GTK theme that matches it closely enough that it becomes low-contrast.

The long-term solution would be to use SVG icons so we can dynamically adjust the color of icons, but that's not going to happen soon.
Flags: needinfo?(shorlander)
Whiteboard: [testday-20131206] → [testday-20131206][Australis:P-]
Bug 764299 should have made SVG icons viable. What else is missing?
Component: Toolbars and Customization → Theme
I believe at least part of the problem was that it seemed like introducing SVG icons would regress us on ts_paint.

I wonder if it's possible to persist the cached SVG from session to session, and only invalidate it if we detect a system theme change?
How does using SVG fix the part where we need to determine the background color in order to pick colors for the icons?
(In reply to :Gijs Kruitbosch from comment #7)
> How does using SVG fix the part where we need to determine the background
> color in order to pick colors for the icons?

It would cover the system theme case because we have corresponding system to choose from. e.g. -moz-appearance:menubar / background-color: -moz-MenuBarText

So we wouldn't have to determine anything.
Flags: needinfo?(shorlander)
Depends on: 572485
See Also: → 994623
Depends on: 994623
Depends on: 1012629
Blocks: 874674
Keywords: regression
Depends on: 637608
After bug 994623 and bug 637608, the all tabs button is the remaining affected piece. Images and CSS adopted from the Windows theme.
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attachment #8443577 - Flags: review?(mdeboer)
Marco, could you please add this to the current iteration?
Points: --- → 2
Flags: needinfo?(mmucci)
Whiteboard: [testday-20131206][Australis:P-] → [qa+]
Added to Iteration 33.1
Points: 2 → ---
Flags: needinfo?(mmucci)
Whiteboard: [qa+] → p=2 s=33.1 [qa+]
Iteration: --- → 33.2
Points: --- → 2
QA Whiteboard: [qa+]
Whiteboard: p=2 s=33.1 [qa+]
Flags: firefox-backlog+
QA Contact: cornel.ionce
See Also: 994623
Comment on attachment 8443577 [details] [diff] [review]
remaining piece: the all tabs button

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

Alright, looks good to me!

Thanks Dão.

Apology for the review delay... many things on my plate atm.
Attachment #8443577 - Flags: review?(mdeboer) → review+
https://hg.mozilla.org/mozilla-central/rev/4125eff7a185
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
Mozilla/5.0 (X11; Linux x86_64; rv:33.0) Gecko/20100101 Firefox/33.0

Verified this issue as fixed using latest Firefox Nightly, build ID: 20140629030206 on Ubuntu 13.04 64bit.
Status: RESOLVED → VERIFIED
QA Whiteboard: [qa+] → [qa!]
You need to log in before you can comment on or make changes to this bug.