Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 383447

Summary:	[ScrolledComposite][List] Mousewheel scrolling broken in firefox 13
Product:	[RT] RAP	Reporter:	Tim Buschtoens <tbuschto>
Component:	RWT	Assignee:	Project Inbox <rap-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	tbuschto
Version:	1.5
Target Milestone:	2.0 M1
Hardware:	All
OS:	All
Whiteboard:	sr151

Description Tim Buschtoens

2012-06-25 10:00:21 EDT

From 13 on smooth scrolling is enabled by default. For widgets using native browser scrolling (List and ScrolledComposite) this break scrolling with mousewheel. Apparently a scroll event is fired by the browser several times per "smooth) scroll animation. However, when the dom is accessed within such an event (e.g. setting the scrollbar thumb), this will stop the scrolling-animation.

Comment 1 Tim Buschtoens

2012-06-25 12:10:42 EDT

Fixed with commit c5271d42bc6ed523f382a14cfc4a939810bd3738.

It was actually not the scrollbar rendering directly, but the event caused by the scrollbar that synced it to the client area. That caused the scrolling to stop since firefox (correctly) gives a programatic set scroll position a higher priority than the running "smooth" animation. Fixed by camparing the new with the old value before actually applying it.

Comment 2 Tim Buschtoens

2012-06-25 12:12:19 EDT

I recommand this for the service release, since its an uncritical change and a pretty annoying bug.

Comment 3 Ralf Sternberg

2012-06-28 12:15:19 EDT

I think this is safe to apply to 1.5.

Comment 4 Ivan Furnadjiev

2012-09-07 03:32:59 EDT

Backport the fix to 1.5 maintenance branch with commit 94bcfae6dad1c9c7f690efb02f04d5aec01fbffe.