Ensure AppMenu panel position relative to the anchor matches the specification
Categories
(Firefox :: Menus, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox89 | --- | fixed |
People
(Reporter: mconley, Assigned: emmamalysz)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proton-hamburger-menu][priority:2a])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Spinning this out from bug 1694976.
amylee writes: "The position of the menu is too high and too far to the right of the screen (please reference spec for position)"
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 1•4 years ago
|
||
From my PM review, I immediately noticed this behavior and it appears to be different between platforms (and maybe themes). On Windows 10 w/ Light Theme it appears correctly, but on OS X w/ Dark Theme I see this defect. The AppMenu should be right-aligned to the edge of the hamburger button and the top of the menu should start after the bottom of the hamburger button. Right now on macOS it's aligned to the right edge of the browser window and covers up the bottom 6th of the hamburger button. It stands out in a not great way, I'd like to see this fixed if we have time during polish before shipping.
-> priority:2a
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Reporter | ||
Comment 2•4 years ago
|
||
Thanks for looking at this, emalysz! Just a heads up that there are some CSS rules that make these panels behave differently on macOS than on Windows and Linux. Specifically here: https://searchfox.org/mozilla-central/rev/05cea55db68db7a3c8531cadaa169e46176e3983/toolkit/themes/osx/global/popup.css#34-42
Also, a number of CSS rules have been added to nudge panels back and forth a bit: https://searchfox.org/mozilla-central/rev/05cea55db68db7a3c8531cadaa169e46176e3983/browser/themes/shared/customizableui/panelUI.inc.css#208-263
And those might need adjusting with Proton enabled.
Assignee | ||
Comment 3•4 years ago
|
||
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 5•4 years ago
|
||
bugherder |
Reporter | ||
Updated•4 years ago
|
Description
•