Off-by-one-pixel error in findbar/devtools letterboxing patch
Categories
(Core :: Window Management, defect, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox69 | --- | fixed |
People
(Reporter: tjr, Assigned: timhuang)
References
Details
Attachments
(1 file)
howaboutsynergy posted a patch at https://bugzilla.mozilla.org/show_bug.cgi?id=1546832#c25 that needs investigating and review.
fyi, iirc, findBar.length is 1 after the first time the find bar is opened, and it remains 1 even if it gets closed afterwards.
The other issue is that the +1 to devtoolsOffset isn't needed, else the size is like 1001.
both ^ are true for me anyway and I've used [1] to test, but I suppose [2] can also be used.
[1] https://www.whatismybrowser.com/detect/how-big-is-my-web-browser
[2] https://ghacksuserjs.github.io/TorZillaPrint/TorZillaPrint.html#screen
|
Assignee | |
Comment 2•5 years ago
|
||
Thanks howaboutsynergy,
The findBar.length remains 1 after the findBar is closed since it is hidden, but not removed. And it seems there is no separator(or it is inside the iframe) for devtools iframe, so we don't need this one extra pixel according to my test.
I have a patch for this.
|
|
Assignee | |
Comment 3•5 years ago
|
||
Pushed by tihuang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a65d8c098c9a Fixing the off-by-one-pixel issue of find bar and devtool iframe for Letterboxing. r=johannh
|
||
Comment 5•5 years ago
|
||
| bugherder | ||
Description
•