Closed
Bug 1061674
Opened 7 years ago
Closed 7 years ago
[Browser] Blurry and Misaligned SSL Lock
Categories
(Firefox OS Graveyard :: Gaia::System::Browser Chrome, defect)
Firefox OS Graveyard
Gaia::System::Browser Chrome
x86
Gonk (Firefox OS)
Tracking
(b2g-v2.0 unaffected, b2g-v2.1 fixed, b2g-v2.2 fixed)
RESOLVED
FIXED
2.1 S4 (12sep)
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | unaffected |
b2g-v2.1 | --- | fixed |
b2g-v2.2 | --- | fixed |
People
(Reporter: epang, Assigned: kgrandon)
References
Details
(Whiteboard: [systemsfe])
Attachments
(5 files, 1 obsolete file)
Hey Kevin, Feel free to redirect this bug, I wasn't sure who to assign it to. The ssl icon looks blurry in the chrome, while other icons look sharp. Can you take a look and let me know if it's a visual asset problem? Also, the icon should take up 30 px of horizontal space, so the space is equal on both sides of the icon. Here's the spec for reference: https://mozilla.box.com/s/oedme1y7u6m3s6lxvk0a Thanks!
Reporter | ||
Updated•7 years ago
|
Component: Gaia::Browser → Gaia::System::Browser Chrome
Reporter | ||
Comment 1•7 years ago
|
||
(In reply to Eric Pang [:epang] from comment #0) > Created attachment 8482716 [details] > Blurry and missaligned SSL Lock > > Hey Kevin, > > Feel free to redirect this bug, I wasn't sure who to assign it to. > > The ssl icon looks blurry in the chrome, while other icons look sharp. Can > you take a look and let me know if it's a visual asset problem? > > Also, the icon should take up 30 px of horizontal space, so the space is > equal on both sides of the icon. > > Here's the spec for reference: > https://mozilla.box.com/s/oedme1y7u6m3s6lxvk0a > > Thanks! Sorry, looks like the visual assets are too small, fixing them now :).
Reporter | ||
Comment 2•7 years ago
|
||
(In reply to Eric Pang [:epang] from comment #1) > (In reply to Eric Pang [:epang] from comment #0) > > Created attachment 8482716 [details] > > Blurry and missaligned SSL Lock > > > > Hey Kevin, > > > > Feel free to redirect this bug, I wasn't sure who to assign it to. > > > > The ssl icon looks blurry in the chrome, while other icons look sharp. Can > > you take a look and let me know if it's a visual asset problem? > > > > Also, the icon should take up 30 px of horizontal space, so the space is > > equal on both sides of the icon. > > > > Here's the spec for reference: > > https://mozilla.box.com/s/oedme1y7u6m3s6lxvk0a > > > > Thanks! > > Sorry, looks like the visual assets are too small, fixing them now :). the ssl lock and broken ssl can both be found with the browser chrome icons https://mozilla.box.com/s/xleyd4fnpl5e2gvo2bxr They are resized to the spec now, hopefully this fixes the blurriness and margins, thanks!
Assignee | ||
Comment 3•7 years ago
|
||
Assignee | ||
Comment 4•7 years ago
|
||
Hey Eric - Could you take a look at this screenshot on a flame? I've centered the icon within a 3rem box, just wanted to make sure this is what you were looking for. Thanks!
Attachment #8482833 -
Flags: ui-review?(epang)
Reporter | ||
Comment 5•7 years ago
|
||
(In reply to Kevin Grandon :kgrandon from comment #4) > Created attachment 8482833 [details] > Screenshot - Implementation on flame > > Hey Eric - Could you take a look at this screenshot on a flame? I've > centered the icon within a 3rem box, just wanted to make sure this is what > you were looking for. Thanks! Hey Kevin, looks like i made the icons too small. I've updated them on box again. https://mozilla.box.com/s/xleyd4fnpl5e2gvo2bxr Sorry for all the back and forth. I think it should be good after this, I'll look to ui-review tomorrow morning. Thanks again for working on this!
Flags: needinfo?(kgrandon)
Assignee | ||
Comment 6•7 years ago
|
||
Uploading a zip of the icons here in case they are needed for the future.
Flags: needinfo?(kgrandon)
Assignee | ||
Updated•7 years ago
|
Attachment #8482833 -
Attachment is obsolete: true
Attachment #8482833 -
Flags: ui-review?(epang)
Assignee | ||
Comment 7•7 years ago
|
||
Comment on attachment 8482832 [details] [review] Pull request - Update SSL Icon assets Hey Eric - assets have been updated now. Since this is mainly just an asset swap, would you mind flipping the review? flag here instead of ui-review if it looks good to you? I'll try to provide a screenshot as well here shortly.
Attachment #8482832 -
Flags: review?(epang)
Assignee | ||
Comment 8•7 years ago
|
||
Reporter | ||
Comment 9•7 years ago
|
||
(In reply to Kevin Grandon :kgrandon from comment #8) > Created attachment 8483773 [details] > Screenshot of implementation The icon looks good now, just one more nit pick.. is it possible to horizontally center the icon between the edge of the input field and where the text starts? If possible IRC me when it's done so I can quickly R+ :) Thanks again Kevin!
Flags: needinfo?(kgrandon)
Assignee | ||
Comment 10•7 years ago
|
||
Flags: needinfo?(kgrandon)
Reporter | ||
Comment 11•7 years ago
|
||
Comment on attachment 8482832 [details] [review] Pull request - Update SSL Icon assets Just needs to move 1px right, but you already know what :). R+ Thanks again!
Attachment #8482832 -
Flags: review?(epang) → review+
Assignee | ||
Comment 12•7 years ago
|
||
In master: https://github.com/mozilla-b2g/gaia/commit/1d910d919e778622ef0569f3f6896fae1a609d56
Assignee | ||
Updated•7 years ago
|
Assignee | ||
Updated•7 years ago
|
Target Milestone: --- → 2.1 S4 (12sep)
Assignee | ||
Comment 13•7 years ago
|
||
Comment on attachment 8482832 [details] [review] Pull request - Update SSL Icon assets This is very low-risk visual polish we would like to uplift.
Attachment #8482832 -
Flags: approval-gaia-v2.1?(bbajaj)
Updated•7 years ago
|
Attachment #8482832 -
Flags: approval-gaia-v2.1?(bbajaj) → approval-gaia-v2.1+
Comment 14•7 years ago
|
||
v2.1: https://github.com/mozilla-b2g/gaia/commit/13e2932cba9eada50985a826792405a0a8162489
You need to log in
before you can comment on or make changes to this bug.
Description
•