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

Bug 468780

Summary: The FXPanOnX policies should be based on the FXChangeViewportPolicy.
Product: [Tools] GEF Reporter: Matthias Wienand <matthias.wienand>
Component: GEF MVCAssignee: Matthias Wienand <matthias.wienand>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: unspecified   
Target Milestone: 3.10.1 (Mars SR1) M1   
Hardware: All   
OS: All   
Whiteboard:

Description Matthias Wienand CLA 2015-05-29 07:59:01 EDT
Currently the FXPanOnScrollPolicy and FXPanOnTypePolicy directly change the ViewportModel using an FXChangeViewportOperation. But in order to decouple changing the viewport from the interactions, both of the interaction policies should be based on FXChangeViewportPolicy.

In this context, the FXChangeViewportPolicy has to be extended to support changing the scroll offset.
Comment 1 Matthias Wienand CLA 2015-07-02 09:18:57 EDT
I extended the FXChangeViewportPolicy and FXChangeViewportOperation to allow changing the scroll offset and based the FXPanOnScrollPolicy and FXPanOnTypePolicy on them. The code is published on the master branch, therefore, I resolve this ticket as fixed for 3.10.1M1.