Closed
Bug 1403382
Opened 7 years ago
Closed 7 years ago
Dragging the back button out of the toolbar in customize mode adjusts the toolbar height
Categories
(Firefox :: Toolbars and Customization, defect, P1)
Tracking
()
VERIFIED
FIXED
Firefox 58
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox56 | --- | unaffected |
firefox57 | + | verified |
firefox58 | --- | verified |
People
(Reporter: abenson, Assigned: jaws)
References
Details
(Whiteboard: [reserve-photon-structure])
Attachments
(2 files)
(deleted),
video/quicktime
|
Details | |
(deleted),
text/x-review-board-request
|
Gijs
:
review+
Sylvestre
:
approval-mozilla-beta+
|
Details |
If you drag the back button anywhere, the height of the toolbar shrinks. This doesn't happen in compact mode and it feels much more stable.
Even though you can't remove the back button, the interaction doesn't feel great. A min-height on the toolbar should solve the problem.
Updated•7 years ago
|
Whiteboard: [photon-structure] → [photon-structure] [triage]
Comment 1•7 years ago
|
||
Jared, is this something you could take a look at?
Blocks: 1387313
Flags: needinfo?(jaws)
Updated•7 years ago
|
Priority: -- → P4
Whiteboard: [photon-structure] [triage] → [reserve-photon-structure]
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Flags: needinfo?(jaws)
Assignee | ||
Comment 2•7 years ago
|
||
[Tracking Requested - why for this release]: UI regression in 57 during customize mode.
status-firefox56:
--- → unaffected
status-firefox57:
--- → affected
status-firefox58:
--- → affected
tracking-firefox57:
--- → ?
Updated•7 years ago
|
Flags: qe-verify?
Priority: P4 → P1
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8914591 [details]
Bug 1403382 - Dragging the back button out of the toolbar in customize mode shouldn't affect the toolbar height.
https://reviewboard.mozilla.org/r/185938/#review190976
::: browser/components/customizableui/CustomizeMode.jsm:1869
(Diff revision 1)
> + let toolbarParent = draggedItem.closest("toolbar");
> + if (toolbarParent) {
> + toolbarParent.style.removeProperty("min-height");
> + }
We should probably do this in `_onDragEnd` as well, as not every drag ends with a drop? Unfortunately I don't think we can only do it there, because there are some early exits in `_applyDrop` that will not call it (maybe they should? Hard to tell...).
Attachment #8914591 -
Flags: review?(gijskruitbosch+bugs) → review+
Comment hidden (mozreview-request) |
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2c2f9dc1f79b
Dragging the back button out of the toolbar in customize mode shouldn't affect the toolbar height. r=Gijs
Assignee | ||
Comment 7•7 years ago
|
||
Comment on attachment 8914591 [details]
Bug 1403382 - Dragging the back button out of the toolbar in customize mode shouldn't affect the toolbar height.
Approval Request Comment
[Feature/Bug causing the regression]: bug 1387313
[User impact if declined]: visual glitch while customizing and moving the back button around
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: no
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: no
[Why is the change risky/not risky?]: straightforward javascript change that fixes the height of the toolbar before hiding the button and clears the fixed height after the drag has finished
[String changes made/needed]: none
Attachment #8914591 -
Flags: approval-mozilla-beta?
Comment 8•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Comment 9•7 years ago
|
||
Comment on attachment 8914591 [details]
Bug 1403382 - Dragging the back button out of the toolbar in customize mode shouldn't affect the toolbar height.
Improve photon, taking it.
Attachment #8914591 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 10•7 years ago
|
||
bugherder uplift |
Comment 11•7 years ago
|
||
User-facing bug with no automation coverage and without a fix confirmation in Nightly. Flagging for manual testing.
Flags: qe-verify? → qe-verify+
Updated•7 years ago
|
status-firefox-esr52:
--- → unaffected
Comment 12•7 years ago
|
||
I have reproduced this bug with Nightly 58.0a1 (2017-09-26) on Windows 8, 64-bit.
The bug's fix is now verified on Latest Nightly and Latest Beta
Latest Nightly :
Build ID 20171006100327
User Agent Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:58.0) Gecko/20100101 Firefox/58.0
Latest Beta :
Build ID 20171005195903
User Agent Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0
[bugday-20171004]
Updated•7 years ago
|
Status: RESOLVED → VERIFIED
Comment 13•7 years ago
|
||
I have reproduced the issue mentioned in comment 0, using an affected Firefox 58.0a1 build (BuildId:20170926220106).
I have verified that the issue is not reproducible using Firefox 57.0b6 (Build Id:20171005195903) and Firefox 58.0a1 (BuildId:20171009100134) on Windows 10 64bit, macOS 10.11.6 and Ubuntu 16.04 64bit.
Thanks Anika for the help!
You need to log in
before you can comment on or make changes to this bug.
Description
•