Closed Bug 1271680 Opened 3 years ago Closed 3 years ago
[Windows] previous/next button icons need Hi
Bug 896304 added versions of these icons for OSX but not for Windows, where they are needed too.
Summary: [Windows] magnifying glass icon and previous/next button icons need HiDPI versions → [Windows] previous/next button icons need HiDPI versions
Stephen, can I use the same arrow icons as on OSX (svg) or would you like to provide me with different ones?
Assignee: nobody → mdeboer
Status: NEW → ASSIGNED
(In reply to Mike de Boer [:mikedeboer] from comment #1) > Stephen, can I use the same arrow icons as on OSX (svg) or would you like to > provide me with different ones? Yes, you can use the same icons.
While you are here, why not use them on Linux too?
(In reply to Panos Astithas [:past] from comment #3) > While you are here, why not use them on Linux too? Consider it done.
Iteration: --- → 49.2 - May 23
Points: --- → 2
Comment on attachment 8752166 [details] [diff] [review] Patch v1: use scalable SVG icons for the findbar navigation arrows Hm, should we add an outline to the SVG icon so that it's visible on different backgrounds? It currently seems to use just a single fill color. We could also use -moz-dialogtext as the fill color, but we'd need to take care of lightweight themes too.
Good point! Let's ask Stephen if he wants to adjust the SVG - in case a drop-shadow style outline is preferred - or define a stroke. In any case I'd like to know the color value of this stroke. Lightweight themes are not applied to the findbar.
If lightweight themes aren't a concern, we can just use -moz-dialogtext instead of #4c4c4c, right?
(In reply to Dão Gottwald [:dao] from comment #8) > If lightweight themes aren't a concern, we can just use -moz-dialogtext > instead of #4c4c4c, right? That should probably work. I think we are applying a gradient overlay on those buttons though. So that would probably need to be removed or the effect reduced to avoid potential conflicts.
-moz-dialogtext is rather black on my windows box... is that correct? And if so, is that the intended look?
It's in fact 100% black with default settings on Windows, just like the text in the find bar. Matching the text color sounds right to me.
I agree with Dao, this should be -moz-dialogtext and it will be fine that it is black. As it is right now on Windows 10 they are a dark navy blue which doesn't match any of the other colors in the findbar.
Comment on attachment 8753490 [details] [diff] [review] Patch v2: use scalable SVG icons for the findbar navigation arrows I think this will work fine. If we find that it's too dark, we can still reduce the opacity.
Attachment #8753490 - Flags: review?(dao+bmo) → review+
https://hg.mozilla.org/integration/fx-team/rev/f79314b1a23fa5fd403fbcf97b81a59a800d9614 Bug 1271680 - use scalable SVG icons for the findbar navigation arrows on Windows and Linux too. r=dao
You need to log in before you can comment on or make changes to this bug.