Closed Bug 1400232 Opened 2 years ago Closed 2 years ago
Hide the menu button separator in compact mode
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0 Build ID 20170914100122 [Affected version]: Nightly 57.0a1 [Affected platforms]: Windows 10 x64, Windows 7 x32, Windows 10 x64 Surface Pro4, Ubuntu 16.04 x64, Mac OS x 10.12 [Steps to reproduce]: 1. Launch Nightly 57.0a1 2. Set Density to Compact 3. Observe the browser toolbar area [Expected Result]: When Density is set to Compact, the separation line in front of the hamburger Menu is not displayed. [Actual Result]: The separation line is displayed. [Notes]: - Please see attached screenshot. - Here are the mockups for this feature: https://mozilla.invisionapp.com/share/ENBBK0F9U#/screens/238616821
I think that's correct, it's a detail we missed.
Whiteboard: [photon-visual] → [reserve-photon-visual]
(In reply to Valentina Claudia Ona from comment #0) > [Affected platforms]: Windows 10 x64, Windows 7 x32, Windows 10 x64 Surface > Pro4, Ubuntu 16.04 x64, Mac OS x 10.12 > [Expected Result]: When Density is set to Compact, the separation line in > front of the hamburger Menu is not displayed. I can't reproduce this on Ubuntu and haven't seen it on Windows 10 either, but maybe I'm misunderstanding what this bug is about. Can you attach a screenshot of what you're seeing? Also, can you please update the bug summary to something meaningful?
Summary: An extra appears in compact density → An extra line appears in compact density
Please see the attachment in the description.
Okay, finally understood what this bug is about.
Summary: An extra line appears in compact density → Hide the menu button separator in compact mode
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Priority: P4 → P1
Comment on attachment 8908626 [details] Bug 1400232 - Hide the menu button separator in compact mode. https://reviewboard.mozilla.org/r/180292/#review185440
Attachment #8908626 - Flags: review?(jhofmann) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/925368197441 Hide the menu button separator in compact mode. r=johannh
Is it intentional that the space between the menu button and any icon to it's left is larger than between other icons? It's almost as if you removed the separator but forgot to make the space smaller.
Verified as fixed on Nightly 57.0a1, Build ID 20170917220255 on Windows 10, Windows 10 Surface Pro4, Windows 7, Mac 10.12 and Ubuntu 16.04. I will mark this as fixed verified.
You need to log in before you can comment on or make changes to this bug.