1586091 - Port Bug 1514926 Convert arrowscrollbox binding to a custom element

Status: RESOLVED FIXED

(Thunderbird :: Toolbars and Tabs, task)

Comment 1

[Richard Marti (:Paenglab)]

Attachment: 1586091-arrowscrollbox.patch

Port of https://hg.mozilla.org/mozilla-central/rev/45b2d685bc5f

The weird thing is, that it begins to scroll when the mouse is over the arrow buttons and not only when the buttons are clicked. But this happens already with older Dailies, so not introduced by this patch.

Alessandro, when you find why this happens, could you fix it? If not we can file a new bug for this.

Comment 2

[Alessandro Castellani [:aleca]]

Attachment: 1586091-arrowscrollbox.patch

Thanks for taking care of this.
I updated the patch to fix the following issues:

• Dropdown indicator not appearing/disappearing on resize.
• Drag&Drop not scrolling when hovering over the next/prev buttons.
• Set the scroll only on click, not over.

Here's a try run to see if we butchered some tests: https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=3a71b1addfed465a9db210e2ccdc2579ec2b2480

Comment 3

[Jorg K (CEST = GMT+2)]

Comment on attachment 9098709 [details] [diff] [review]
1586091-arrowscrollbox.patch

I looked at the interdiff and it looks OK to me. I'm landing this now.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/78c57b3a917c
Port bug 1514926: Convert the arrowscrollbox binding to a Custom Element. r=aleca DONTBUILD

Comment 5

[Jorg K (CEST = GMT+2)]

Just in case anyone is wondering: I have a green try with all the stuff landed with DONTBUILD:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=5a5125885a9c6aa5ebc6bfd28fc1bf7cc33baf18

Comment 6

[Richard Marti (:Paenglab)]

Comment on attachment 9098709 [details] [diff] [review]
1586091-arrowscrollbox.patch

Looks a lot better now, thanks.