Closed
Bug 1018720
Opened 10 years ago
Closed 10 years ago
Translation infobar: missing HiDPI icon
Categories
(Firefox :: Theme, defect)
Tracking
()
People
(Reporter: soeren.hentzschel, Assigned: smacleod)
References
Details
(Whiteboard: [translation] )
Attachments
(8 files)
41.38 KB,
image/png
|
Details | |
1.13 KB,
application/zip
|
Details | |
48.55 KB,
image/png
|
florian
:
feedback-
mmaslaney
:
feedback+
|
Details |
3.97 KB,
patch
|
florian
:
review+
|
Details | Diff | Splinter Review |
2.81 KB,
patch
|
florian
:
review+
|
Details | Diff | Splinter Review |
1.57 KB,
patch
|
florian
:
review+
|
Details | Diff | Splinter Review |
2.00 KB,
patch
|
florian
:
review+
|
Details | Diff | Splinter Review |
50.76 KB,
image/png
|
Details |
The arrow icon in the select box needs a HiDPI version.
Updated•10 years ago
|
Flags: firefox-backlog+
Whiteboard: p=1
Comment 1•10 years ago
|
||
Sevaan, do we have a HiDPI version of this icon somewhere?
Flags: needinfo?(sfranks)
Comment 2•10 years ago
|
||
The arrow can be found in the assets mmaslaney has uploaded here: https://bugzilla.mozilla.org/show_bug.cgi?id=1024741#c7
Updated•10 years ago
|
Flags: needinfo?(sfranks)
Comment 3•10 years ago
|
||
(In reply to Sevaan Franks [:sevaan] from comment #2) > The arrow can be found in the assets mmaslaney has uploaded here: > https://bugzilla.mozilla.org/show_bug.cgi?id=1024741#c7 Are we intentionally switching from a double arrow to a single arrow?
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → smacleod
Status: NEW → ASSIGNED
Updated•10 years ago
|
Iteration: --- → 33.2
Points: --- → 1
QA Whiteboard: [qa+]
Whiteboard: p=1
Updated•10 years ago
|
Whiteboard: [translation]
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Florian Quèze [:florian] [:flo] from comment #3) > (In reply to Sevaan Franks [:sevaan] from comment #2) > > The arrow can be found in the assets mmaslaney has uploaded here: > > https://bugzilla.mozilla.org/show_bug.cgi?id=1024741#c7 > > Are we intentionally switching from a double arrow to a single arrow? ^ Sevaan?
Flags: needinfo?(sfranks)
Updated•10 years ago
|
QA Contact: bogdan.maris
Comment 5•10 years ago
|
||
(also, in-content we seem to have hi-dpi spinner-like icons for e.g. select boxes... don't know how hard it is to just steal that, and/or why this is a <menulist> rather than an html:select...)
Comment 6•10 years ago
|
||
Hey Steven, sorry for the late reply. Yes, that is the intention as we move toward a new, more unified style. (In reply to Steven MacLeod [:smacleod] from comment #4) > (In reply to Florian Quèze [:florian] [:flo] from comment #3) > > (In reply to Sevaan Franks [:sevaan] from comment #2) > > > The arrow can be found in the assets mmaslaney has uploaded here: > > > https://bugzilla.mozilla.org/show_bug.cgi?id=1024741#c7 > > > > Are we intentionally switching from a double arrow to a single arrow? > > ^ Sevaan?
Flags: needinfo?(sfranks)
Comment 7•10 years ago
|
||
Michael Maslaney, can you confirm the single arrow vs. double? Flo-retina pointed out that we are still using double-arrows in the Incontent UI stuff.
Flags: needinfo?(mmaslaney)
Comment 8•10 years ago
|
||
We are moving forward with the single arrow, attached.
Flags: needinfo?(mmaslaney)
Assignee | ||
Comment 9•10 years ago
|
||
This is a screenshot using the updated HiDPI icon provided. Is it just me or does it bother anyone else that the options menu uses a different arrow icon? Thoughts?
Attachment #8450511 -
Flags: feedback?(mmaslaney)
Attachment #8450511 -
Flags: feedback?(florian)
Comment 10•10 years ago
|
||
Those arrows should definitely match. A couple other things I can see: - The background colour of the options button should match the language drop-down (there should be a very subtle gradient). - The border of the language drop-down should match the border of the options button: #cdd3d8 - The language drop-down should be the same height as the options button (slightly larger)
Comment 11•10 years ago
|
||
(In reply to Sevaan Franks [:sevaan] from comment #10) > - The language drop-down should be the same height as the options button > (slightly larger) We handled this in bug 1022405.
Assignee | ||
Comment 12•10 years ago
|
||
This patch switches the select box and adds a HiDPI variant. This pretty much brings things to the state of the previously posted screenshot.
Attachment #8451889 -
Flags: review?(florian)
Assignee | ||
Comment 13•10 years ago
|
||
The patch in Bug 1028942 was incorrect. This patch fixes the buttons so that the options dropdown has the correct style as well.
Attachment #8451890 -
Flags: review?(florian)
Assignee | ||
Comment 14•10 years ago
|
||
This makes the styling of the select dropdown match the options dropdown (and fixes the height).
Attachment #8451891 -
Flags: review?(florian)
Assignee | ||
Comment 15•10 years ago
|
||
Finally, this updates the icon of the options dropdown to match the new select dropdown icon introduced in patch 1.
Attachment #8451893 -
Flags: review?(florian)
Assignee | ||
Comment 16•10 years ago
|
||
This is a screenshot of the inforbar after applying the patches. The hover states of all the elements (excluding the "Translate" button) are now consistent as well.
Updated•10 years ago
|
Iteration: 33.2 → 33.3
Comment 17•10 years ago
|
||
That looks great, Steven. Thanks!
Updated•10 years ago
|
Attachment #8451889 -
Flags: review?(florian) → review+
Updated•10 years ago
|
Attachment #8451890 -
Flags: review?(florian) → review+
Updated•10 years ago
|
Attachment #8451891 -
Flags: review?(florian) → review+
Updated•10 years ago
|
Attachment #8451893 -
Flags: review?(florian) → review+
Comment 18•10 years ago
|
||
Comment on attachment 8450511 [details]
screenshot-select-box-icon.png
Next version looks much better..
Attachment #8450511 -
Flags: feedback?(florian) → feedback-
Assignee | ||
Comment 19•10 years ago
|
||
Patch 1: https://hg.mozilla.org/integration/fx-team/rev/d552d2ba30a5 Patch 2: https://hg.mozilla.org/integration/fx-team/rev/533c1369f45b Patch 3: https://hg.mozilla.org/integration/fx-team/rev/b0084d208f32 Patch 4: https://hg.mozilla.org/integration/fx-team/rev/0102658bc570
https://hg.mozilla.org/mozilla-central/rev/d552d2ba30a5 https://hg.mozilla.org/mozilla-central/rev/533c1369f45b https://hg.mozilla.org/mozilla-central/rev/b0084d208f32 https://hg.mozilla.org/mozilla-central/rev/0102658bc570
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
Comment 21•10 years ago
|
||
We don`t have a Mac with Retina Display here, Sören can you please verify that the issue is fixed?
Flags: needinfo?(cadeyrn)
Comment 23•10 years ago
|
||
Thanks a lot, marking this as verified then.
Status: RESOLVED → VERIFIED
QA Whiteboard: [qa+] → [qa!]
Updated•10 years ago
|
Attachment #8450511 -
Flags: feedback?(mmaslaney) → feedback+
You need to log in
before you can comment on or make changes to this bug.
Description
•