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

Bug 493351

Summary:	Simplify ZestContentViewer by using as simple ListChangeListener rather than a SelectionNotifier.
Product:	[Tools] GEF	Reporter:	Alexander Nyßen <nyssen>
Component:	GEF Zest	Assignee:	Alexander Nyßen <nyssen>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3
Version:	0.2.0
Target Milestone:	4.0.0 (Neon) RC1
Hardware:	All
OS:	All
Whiteboard:

Description Alexander Nyßen

2016-05-10 10:40:24 EDT

ZestContentViewer currently uses a local nested SelectionNotifier class, which implements IActivatable. As the notifier is always activated when being registered and always deactivated when being unregistered, we can replace it with a simple listener (that does not implement IActivatable) for the sake of simplicity.

Comment 1 Alexander Nyßen

2016-05-10 10:42:19 EDT

Replaced nested SelectionNotifier class with a simple ListChangeListener on the SelectionModel.

Resolving as fixed in 4.0.0 RC1.