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

Bug 484471

Summary:	NavigationModel.add/removeSkipNextLayout() should not be required.
Product:	[Tools] GEF	Reporter:	Alexander Nyßen <nyssen>
Component:	GEF Zest	Assignee:	gef-inbox <gef-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	matthias.wienand
Version:	0.2.0
Target Milestone:	4.0.0 / 3.11.0 (Neon) M5
Hardware:	All
OS:	All
Whiteboard:

Description Alexander Nyßen

2015-12-16 05:05:54 EST

Instead of skipping layout for certain nodes, we should apply layout only when it is needed, i.e. dependent on changes applied by the user. The layout should not run without an interaction triggering it (so that no need to skip an automatic layout pass is required).

Comment 1 Matthias Wienand

2016-01-15 07:46:08 EST

I refactored the code so that skipping of automatic layout passes is no longer needed. A layout pass is now triggered only when it is needed, i.e. upon layout-bounds changes and upon content changes. The code is published on the master branch, therefore, I resolve this ticket as fixed for 3.11.0M5.