Closed
Bug 1417016
Opened 7 years ago
Closed 5 years ago
Location bar overlaps the "tip" notification box
Categories
(Firefox :: Address Bar, defect, P2)
Firefox
Address Bar
Tracking
()
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
firefox57 | --- | unaffected |
firefox58 | --- | fix-optional |
firefox59 | --- | fix-optional |
People
(Reporter: emilghitta, Unassigned)
References
Details
(Keywords: regression, Whiteboard: [fxsearch])
Attachments
(1 file, 1 obsolete file)
(deleted),
image/gif
|
Details |
[Affected versions]:
59.0a1 (BuildId:20171113220112)
58.0b3 (BuildId:20171114032831)
[Unaffected versions]:
57.0 (BuildId:20171112125346)
[Affected platforms]:
Windows 10 64bit.
macOS 10.12.
Ubuntu 16.04 64bit.
[Steps to reproduce]:
1. Launch Firefox with a clean profile.
2. Resize Firefox.
3. Write something inside the location bar.
[Expected result]:
The "tip" notification box is properly displayed.
[Actual result]:
The "tip" notification box is displayed under the location bar.
[Regression range]:
This is a regression:
Last good revision: c5695ec7ac409ad20c0076197cdeb3e792f26ef1
First bad revision: 6d50619818688955eb9dcb980a3b666521a8cf1b
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=c5695ec7ac409ad20c0076197cdeb3e792f26ef1&tochange=6d50619818688955eb9dcb980a3b666521a8cf1b
[Additional notes]:
For further details regarding this issue please observe the attached screencast.
Reporter | ||
Comment 1•7 years ago
|
||
Hi Marco!
Can you please have a look into this?
Thanks!
Blocks: 1367790
Flags: needinfo?(mak77)
Comment 2•7 years ago
|
||
This bug pretty much exists from when we added the new notification, the only reason it's now more evident is that by removing the old notification the panel has shrinked a bit in height. Indeed it's a <deck>, and as such it takes the height of the tallest content, that was the old opt-in notification.
We could probably increase the height of the bar to mitigate it a bit (more padding maybe?).
IIRC the fancy overlapping is due to the mixture of flexboxes we use here (both xul and html), that is necessary to keep the text visible in such small windows.
P2 because it sounds like the solution could be a simple oneline of css.
Flags: needinfo?(mak77)
Priority: -- → P2
Whiteboard: [fxsearch]
Updated•7 years ago
|
Reporter | ||
Updated•7 years ago
|
Attachment #8928113 -
Attachment is obsolete: true
Reporter | ||
Comment 3•7 years ago
|
||
Comment 4•5 years ago
|
||
this was removed with the Quantum Bar, so the bug is no more valid.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•