159346 - Scrollbar not working when moving mouse out and in again

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[jernej srebrnic]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.1b) Gecko/20020724
BuildID:    2002072408

when you press the up or down arrow of the scroller bar it starts scrolling.
whit the mouse buton pressed move of the arrow and then reenter. the scroler
shoud scroll agein, but it doesn't

Reproducible: Always
Steps to Reproduce:
1. press the up or down arrow of the scroller bar
2. mouse button pressed move of the arrow
3. reenter whit the mouse button still pressed


Actual Results:  the scroller does not scroll

Expected Results:  should scroll when you reenter

the arrow is repainted in hte raight way. when you reeenter it presed, but with
no effect...

Comment 1

[Vidar Haarr (not reading bugmail)]

Confirmed on WinXP trunk build 2002072408.

-> XP Toolkit/Widgets

Comment 2

[Timwi]

*** Bug 227716 has been marked as a duplicate of this bug. ***

Comment 3

[Timwi]

This bug is now over 4 years old and it is still an issue in Firefox 3 beta 5. When will this finally be addressed?

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Part.1 Scrollbar button should capture mouse events and do scroll only when cursor is on it

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Part.2 Repeat scroll even if mouse cursor is outside of the pressed scrollbar button on gtk2 and qt

Comment 6

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Part.3 Add new tests

Comment 7

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Note that the patches cannot emulate Mac's native behavior perfectly. On Mac, all scrollbar buttons of a scrollbar capture mouse events. When you move mouse cursor to the other scrollbar button, it scrolls the other direction. We should separate the bug for Mac later.

Comment 8

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Roc:

Enn has reviewed the patch already. I'd like you to review the widget part.

Comment 9

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Comment on attachment 574754 [details] [diff] [review]
Part.2 Repeat scroll even if mouse cursor is outside of the pressed scrollbar button on gtk2 and qt

Looks like roc also +'ed to the patch accoding to the bug history but I'm not sure it. Roc, did you +'ed for it? Or just changed the reviewer to Enn?

Comment 10

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I r+ed it.

Comment 11

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Okay, thank you. I land them.

Comment 12

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/08a6c9818f93
https://hg.mozilla.org/integration/mozilla-inbound/rev/c6706ccc5a82
https://hg.mozilla.org/integration/mozilla-inbound/rev/14327685e15b

try:
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=84f711b6b2bd

Comment 13

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/14327685e15b
https://hg.mozilla.org/mozilla-central/rev/c6706ccc5a82
https://hg.mozilla.org/mozilla-central/rev/08a6c9818f93