Open
Bug 1816404
Opened 1 year ago
Updated 1 year ago
The overflow menu button has no padding on the left side if the browser is resized to a minimum width while having Touch Density active
Categories
(Firefox :: Theme, defect)
Tracking
()
NEW
| Tracking | Status | |
|---|---|---|
| firefox-esr102 | --- | unaffected |
| firefox110 | --- | wontfix |
| firefox111 | --- | wontfix |
| firefox112 | --- | wontfix |
| firefox113 | --- | fix-optional |
| firefox114 | --- | fix-optional |
People
(Reporter: atrif, Unassigned)
Details
(Keywords: regression)
Attachments
(2 files)
resize_touch.gif
Found in
- 111.0a1 (20230212085204)
Affected versions
- 111.0a1 (20230212085204)
Tested platforms
- Affected platforms: Windows 10x64, Ubuntu 20.04
- Unaffected platforms: macOS 12
Steps to reproduce
- Open Firefox and Customize menu..
- Enable Touch Density and exit customize menu.
- Resize the window horizontally making it smaller and observe the toolbar.
Expected result
- All toolbar items are correctly displayed.
Actual result
- Overflow menu button has no padding on the left side.
Regression range
- Last good revision: a4a8521b307275498ecda987249e72150c3c110d
First bad revision: 0645b630915e38a3a37d076aa7f431a72387699e
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=a4a8521b307275498ecda987249e72150c3c110d&tochange=0645b630915e38a3a37d076aa7f431a72387699e
I don’t know which of the issue is the regressor. Adding Emilio to the CC list since he’s the author of both patches.
Additional notes
- Attached a screen recording.
|
||
Updated•1 year ago
|
|
||
Updated•1 year ago
|
|
||
Comment 1•1 year ago
|
||
I would guess that this is a regression from bug 1795260.
status-firefox114:
--- → fix-optional
Flags: needinfo?(emilio)
|
||
Comment 2•1 year ago
|
||
Well my patch changed the direction of the overflow. Before my patch the hamburger menu overflowed out of view. So this was likely introduced by the extensions button work.
Flags: needinfo?(emilio) → needinfo?(wdurand)
|
||
Comment 3•1 year ago
|
||
The extensions button has been slightly re-positioned on the navbar recently.
:atrif are you still able to reproduce in Nightly?
Flags: needinfo?(wdurand) → needinfo?(alexandru.trif)
|
|
||
Comment 4•1 year ago
|
||
Nevermind, I can reproduce now with touch density.
Flags: needinfo?(alexandru.trif)
You need to log in
before you can comment on or make changes to this bug.
Description
•