Bookmarks Menu(incl. Bookmarks Toolbar) and Bookmarks Widget popup scroll does not work with mouse wheel
Categories
(Toolkit :: UI Widgets, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox-esr68 | --- | unaffected |
firefox68 | --- | unaffected |
firefox69 | --- | unaffected |
firefox70 | --- | verified |
firefox71 | --- | verified |
People
(Reporter: alice0775, Assigned: surkov)
References
(Regression)
Details
(Keywords: nightly-community, regression)
Attachments
(1 file)
Reproducible: always
Steps To Reproduce 1:
- Enable Menubar and Bookmarks Toolbar
- Click Bookmarks menu and/or sub-folder
- Try scroll it with Mouse wheel
Steps To Reproduce 2:
- Customize toolbar and place Bookmarks Widget into toolbar
- Click Bookmarks Widget and/or sub-folder
- Try scroll it with Mouse wheel
Actual Results:
The popup would not scroll with Mouse wheel
Expected Results:
Scroll properly
Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=8bd696e50a12db4f6be4ded519ef4f97ed6fed60&tochange=bd7cd9d7b6ade7bf9fa8441a8686f19c36f2a7e5
Regressed by:
bd7cd9d7b6ade7bf9fa8441a8686f19c36f2a7e5 Alexander Surkov — Bug 1539651 - Convert places-popup-base and places-popup-arrow bindings to Custom Elements r=mak
Reporter | ||
Updated•5 years ago
|
Comment 4•5 years ago
|
||
Same here, also the smooth scrolling effect doesn't work anymore even when scrolling with the on-hover scrollbuttons. It just jumps up or down by a whole menuitem's height.
Assignee | ||
Comment 5•5 years ago
|
||
One problem I can see here is arrowscrollbox cannot find its scrollable children, thus it fails, for example, update scrollup/down buttons. I suspect this may be the only one problem we have to fix to close this bug (and hopefully bug 1565318 as well).
Assignee | ||
Comment 6•5 years ago
|
||
(In reply to alexander :surkov (:asurkov) from comment #5)
One problem I can see here is arrowscrollbox cannot find its scrollable children, thus it fails, for example, update scrollup/down buttons. I suspect this may be the only one problem we have to fix to close this bug (and hopefully bug 1565318 as well).
the second problem is arrowscrollbox scroll buttons styling is wrong, because the styles are defined in global.css, so fixing that in places-menupopup custom element will help
Assignee | ||
Comment 7•5 years ago
|
||
Assignee | ||
Comment 8•5 years ago
|
||
I don't have a mouse on my working machine, so cannot reproduce the original steps to reproduce. I think though the patch should help the original problem, so would be great to have somebody to confirm.
Assignee | ||
Updated•5 years ago
|
Pushed by asurkov@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8e0b9482a087 arrowscrollbox doesn't work properly if hosted in shadow DOM, r=dao
Comment 10•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Comment 11•5 years ago
|
||
I didn't manage to reproduce the bug using the STR provided in the first comment on Fx 70.0a1 (2019-08-08) using Windows 10 x64 and x32. Could you please please confirm that the issue is fixed ?
Reporter | ||
Comment 12•5 years ago
|
||
You should use 09-Sep-2019 Nightly(which is including the offending patch) to reproduce.
I can reproduce the issue on Nightly70.0a1 Build ID 20190809095611.
And verified fixed on Firefox70.0b9 and Nightly71.0a1 Build ID 20190924215237.
Comment 13•5 years ago
|
||
Hello I checked again this issue and I managed to reproduce the bug and also verify that is fixed using the builds suggested by you.
Updated•2 years ago
|
Description
•