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

Bug 491681

Summary: Viewport is not correctly restored when navigating back to a parent graph.
Product: [Tools] GEF Reporter: Matthias Wienand <matthias.wienand>
Component: GEF ZestAssignee: Alexander Nyßen <nyssen>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: nyssen
Version: 0.2.0   
Target Milestone: 4.0.0 (Neon) RC3   
Hardware: All   
OS: All   
Whiteboard:

Description Matthias Wienand CLA 2016-04-14 07:41:09 EDT
When zooming into a nesting node and then navigating back to the parent graph, the viewport is not correctly restored, but instead, it is reset to fit the whole parent graph. It should be restored to the viewport before navigating into the nested graph, i.e. zoomed into the nesting node.
Comment 1 Alexander Nyßen CLA 2016-05-29 11:00:21 EDT
I pushed the following changes to origin/master:

- Enhanced FXChangeViewportOperation with methods that allow setting the initial state (which is needed to update it when navigating into a nested graph).
- Ensured GraphLayoutBehavior does not manipulate the ViewportModel (behaviors should not change viewer model states).
- Ensured that NavigateOperation properly updates the ViewportModel via a nested operation, so it can be undone/redone.
- Ensured SemanticZoomPolicy clears its references to NavigationModel and ContentModel within commit() (as its initialized during init()).

Resolving as fixed in 4.0.0 RC3.