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

Bug 314763

Summary: Implement a non-native client ScrollBar
Product: [RT] RAP Reporter: Ralf Sternberg <rsternberg>
Component: RWTAssignee: Project Inbox <rap-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: ruediger.herrmann, tbuschto
Version: 1.3   
Target Milestone: 1.4 M4   
Hardware: All   
OS: All   
Whiteboard: plan-version=1.4 plan-theme=widgets plan-status=proposed
Bug Depends on: 313034, 326751, 326752, 327240, 330011, 331559    
Bug Blocks: 318293, 323032, 323984, 327046, 329889, 330225    
Attachments:
Description Flags
Fix none

Description Ralf Sternberg CLA 2010-05-27 15:40:41 EDT 
Currently, a couple of RAP widgets use browser-native scrollbars (Tree, Text, ScrolledComposite, Scrollbar). These scrollbars do not fit into the new designs.
We should implement a non-native ScrollBar that is themeable.
Together with this change, RAP widgets that currently use browser-native scrollbars should also make use of the new scrollbar to avoid an inconsistent look.
Comment 1 Rüdiger Herrmann CLA 2010-06-02 13:45:16 EDT 
Created attachment 170845 [details]
Fix
Comment 2 Tim Buschtoens CLA 2010-08-30 11:49:39 EDT 
This will also be helpful for ipad-support, as the native scrollbars of mobile webkit are unuseable with our current Scrollbar-widget.
Comment 3 Tim Buschtoens CLA 2010-10-01 05:06:05 EDT 
We should first refactor widgets that are not yet using the ScrollBar Widget to do so. This includes the List and ScrolledComposite Widget, for which i will open bugs. The Text-Widget also uses a native ScrollBar, but i'm currently not sure that it is possible to replace it without massive changes to the widget itself, so this will remain untouched for now. The same goes for the browser widget.
Comment 4 Tim Buschtoens CLA 2010-11-10 07:53:21 EST 
I created a separate bug for theming the ScrollBar (Bug 329889), and changed Bug 323984 (drag-support on iPad) to include the ScrollBar. Now we can first create a simple replacement for the current ScrollBar before attacking the more advanced issues.
Comment 5 Tim Buschtoens CLA 2010-11-25 11:10:54 EST 
Remember to update Bug 330372 after fixing this one.
Comment 6 Tim Buschtoens CLA 2010-12-02 09:57:27 EST 
Fixed in CVS HEAD.