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)

x86
Gonk (Firefox OS)
defect
Not set
normal

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!
Component: Gaia::Browser → Gaia::System::Browser Chrome
(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 :).
(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!
Attached image Screenshot - Implementation on flame (obsolete) —
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)
(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)
Uploading a zip of the icons here in case they are needed for the future.
Flags: needinfo?(kgrandon)
Attachment #8482833 - Attachment is obsolete: true
Attachment #8482833 - Flags: ui-review?(epang)
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)
(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)
Flags: needinfo?(kgrandon)
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+
In master: https://github.com/mozilla-b2g/gaia/commit/1d910d919e778622ef0569f3f6896fae1a609d56
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S4 (12sep)
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)
Attachment #8482832 - Flags: approval-gaia-v2.1?(bbajaj) → approval-gaia-v2.1+
You need to log in before you can comment on or make changes to this bug.