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
QA Contact: ovidiu.boca
Whiteboard: [photon-visual] → [reserve-photon-visual]
https://hg.mozilla.org/integration/mozilla-inbound/rev/0fa898b4095ea0516113edf55ff60bbf28542b17 Bug 1391331 - Make space between bookmark item icon and label 4px instead of 5px. r=johannh
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/0fa898b4095e Make space between bookmark item icon and label 4px instead of 5px. r=johannh
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.
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.
a year ago
Depends on: 1391625
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
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
Any news about this issue?
Stephen, can you please help me with this, see comment 7. Thanks
(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.
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
You need to log in before you can comment on or make changes to this bug.