Closed
Bug 1457763
Opened 6 years ago
Closed 6 years ago
regression: incorrectly placed autoscroll icon
Categories
(Core :: XUL, defect, P1)
Core
XUL
Tracking
()
VERIFIED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox-esr60 | --- | unaffected |
firefox59 | --- | unaffected |
firefox60 | --- | unaffected |
firefox61 | blocking | verified |
People
(Reporter: jan, Assigned: enndeakin)
References
Details
(Keywords: nightly-community, regression)
Attachments
(3 files)
Debian Testing, KDE, Xorg, Radeon RX480, 2560x1440 Gtk3 Breeze Theme libgtk-3-0:amd64 3.22.29-3 KDE Plasma 5.12.4 KDE Framework 5.44.0 QT 5.10.1 Actual: The autoscroll icon shows up about a centimeter below the cursor. When snapping Nightly to the right half of the window the autoscroll icon is not only a few pixels too far down, but also on the outer right side, directly next to the scrollbar. Only the autoscroll icon is affected. Context menu and <select> dropdowns seem to be fine. Expected: The center of the autoscroll icon should be where the cursor points to. mozregression --good 2018-04-28 --bad 2018-04-29 --pref browser.tabs.drawInTitlebar:false general.autoScroll:true startup.homepage_welcome_url:'https://www.mozilla.org/de/' > 6:49.44 INFO: Last good revision: f846639066aa8f7e5aff7fa363a450885fc887b4 > 6:49.44 INFO: First bad revision: 19b49df2389f777dd45779d89a3f83046c40b3d6 > 6:49.44 INFO: Pushlog: > https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=f846639066aa8f7e5aff7fa363a450885fc887b4&tochange=19b49df2389f777dd45779d89a3f83046c40b3d6 > 19b49df2389f Neil Deakin — Bug 1446961, move PopupBoxObject to XULPopupElement, a new subclass of XULElement. Remove popup.xml methods, r=paolo,bz > 12e768652a88 Neil Deakin — Bug 1446961, add two special cases in PopupBoxObject as supported in popup.xml, r=paolo,bz > 46d7f3cc7102 Neil Deakin — Bug 1446961, add dictionary second argument directly to PopupBoxObject::OpenPopup as supported in popup.xml#openPopup, r=bz > 962c081b1314 Neil Deakin — Bug 1446961, remove unused popup frame methods now that showPopup has been removed, r=paolo > 5659ad69e145 Neil Deakin — Bug 1446961, remove deprecated showPopup method of PopupBoxObject as well as unused GetPopupSetFrame method, and move some methods' positions to group related methods together better, r=paolo,bz > 5be83a2594fb Neil Deakin — Bug 1446961, restructuring to allow nsXULElement to be subclassed. Rename nsXULElement::Create to make it clearer it creates from the prototype element, r=bz > e872787876a5 Neil Deakin — Bug 1446961, remove obsolete calls to showPopup and replace usages of the popup box object with the same methods defined on popups, r=paolo
Reporter | ||
Comment 1•6 years ago
|
||
(FYI: bug 1457691: After maximizing the window, also context menu and <select> dropdowns are misplaced. This can be fixed by pressing F11 twice.)
Reporter | ||
Comment 2•6 years ago
|
||
Mozregression points to your changes. Could you take a look, please? Win10 has the same regression range.
Has Regression Range: --- → yes
Flags: needinfo?(enndeakin)
OS: Linux → All
Hardware: x86_64 → All
This is worse on RTL locales- there, the autoscroll icon is located at the far left of the screen. See attached.
Comment 5•6 years ago
|
||
This sounds pretty bad for people experiencing it.
tracking-firefox61:
--- → +
Assignee | ||
Comment 6•6 years ago
|
||
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Flags: needinfo?(enndeakin)
Attachment #8972344 -
Flags: review?(paolo.mozmail)
Comment 7•6 years ago
|
||
Comment on attachment 8972344 [details] [diff] [review] autoscroll popup should be opened with openPopupAtScreen Thanks!
Attachment #8972344 -
Flags: review?(paolo.mozmail) → review+
Pushed by neil@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/76855675e353 regression, autoscroll popup should be opened with openPopupAtScreen, r=paolo
Updated•6 years ago
|
Comment 10•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/76855675e353
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Reporter | ||
Comment 11•6 years ago
|
||
(In reply to Pulsebot from comment #9) > Pushed by neil@mozilla.com: > https://hg.mozilla.org/integration/mozilla-inbound/rev/76855675e353 > regression, autoscroll popup should be opened with openPopupAtScreen, r=paolo Nightly 61 x64 20180502220059 de_DE 2d83e1843241d869a2fc5cf06f96d3af44c70e70 @ Debian Testing, KDE, Radeon RX480 Thank you! The autoscroll icon is now correctly placed in a maximized and non-maximized window with browser.tabs.drawInTitlebar=true and false. Offtopic: The misplaced bookmarks menu is tracked in bug 1457110.
Status: RESOLVED → VERIFIED
Comment 12•6 years ago
|
||
Managed to reproduce the issue on the Nightly (build ID: 20182502100112 / 2018-05-02) on Windows 10 x64, Mac OS X 10.12 and Ubuntu 16.04 x64. In the latest Nightly (including builds with RTL locales) the issue cannot be reproduced on Windows 10 x64, Mac OS X 10.12 and Ubuntu 16.04 x64, thus I will close the issue as VERIFIED FIXED.
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•