Spacing between Bookmark item Icon and Label should be 4px

VERIFIED FIXED in Firefox 57

Status

()

P1
normal
VERIFIED FIXED
a year ago
a year ago

People

(Reporter: shorlander, Assigned: shorlander)

Tracking

(Blocks: 1 bug)

57 Branch
Firefox 57
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 verified, firefox58 verified)

Details

(Whiteboard: [reserve-photon-visual])

Attachments

(3 attachments)

Created attachment 8898389 [details] [diff] [review]
bookmark-icon-spacing.patch
Attachment #8898389 - Flags: review?(jhofmann)
Comment on attachment 8898389 [details] [diff] [review]
bookmark-icon-spacing.patch

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

Ah, I forgot about this one. Thank you!
Attachment #8898389 - Flags: review?(jhofmann) → review+
Status: NEW → ASSIGNED
Iteration: --- → 57.2 - Aug 29
Flags: qe-verify+
QA Contact: ovidiu.boca
Whiteboard: [photon-visual] → [reserve-photon-visual]

Comment 3

a year ago
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0fa898b4095e
Make space between bookmark item icon and label 4px instead of 5px. r=johannh

Comment 4

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/0fa898b4095e
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox57: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Created attachment 8898793 [details]
Screen Shot 2017-08-18 at 5.03.41 PM.png

I verified this on Windows 10 with the latest Nightly 57.0a1(2017-08-18) and I see 6 px not 4. Please see the attachment and also note that on Mac Os X 10.10 I see 4 px as expected.
Flags: needinfo?(shorlander)
(Assignee)

Comment 6

a year ago
Created attachment 8899013 [details]
Bookmark Item Measurements

There will be some visual difference depending on what the font is and what the end character is. In this case we would have to trust the calculations from the Browser Toolbox or draw a background on the label to use for measuring.

It looks correct to me. On macOS at least.
Flags: needinfo?(shorlander)
Thanks Stephen, you are right on Mac I can confirm that the spacing is 4 px, but I think this 4 px spacing should be applied on all OSes, right? From what I see on Windows this spacing is not the expected one. Please tell me what are the expected results in order to be able to mark this bug as verified or not. Thanks
Blocks: 1391625
No longer depends on: 1391625
Is this bug only for Mac OS? Please tell me the expected result in order to verify this, or the expected result that was described in comment 7 is the right one?  Thanks
Flags: needinfo?(shorlander)
Stephen, can you please help me with this, see comment 7. Thanks
(Assignee)

Comment 11

a year ago
(In reply to ovidiu boca[:Ovidiu] from comment #10)
> Stephen, can you please help me with this, see comment 7. Thanks

According to the inspector the current spacing is 4px and is correct.
Flags: needinfo?(shorlander)
Thanks for your help, I verified this on Mac Os X 10.12 and Windows 10 with FF Nightly 58.0a1(2017-11-02) and I can confirm this fix.
Status: RESOLVED → VERIFIED
status-firefox57: fixed → verified
status-firefox58: --- → verified
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.