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

Bug 331290

Summary: ContextButtonPadData extensibility
Product: [Modeling] Graphiti Reporter: Volker Wegert <eclipse>
Component: CoreAssignee: Project Inbox <graphiti-inbox>
Status: CLOSED FIXED QA Contact:
Severity: minor    
Priority: P3 CC: michael.wenz, tim.kaiser
Version: 0.7.0Flags: michael.wenz: indigo+
Target Milestone: 0.8.0   
Hardware: All   
OS: All   
Whiteboard: Indigo M5 theme_round_offs

Description Volker Wegert CLA 2010-11-28 16:39:54 EST
Build Identifier: 0.7.0

IContextButtonPadData has the annotations
 * @noimplement This interface is not intended to be implemented by clients.
 * @noextend This interface is not intended to be extended by clients.
set. The only provided implementation class DefaultContextButtonPadData has the annotations 
 * @noextend This class is not intended to be subclassed by clients.
set. This is confusing for new users - if I want to add some functionality, e. g. change the pad location, I have to either implement the interface or extend the class, both of which is discouraged by the annotations.

Reproducible: Always
Comment 1 Michael Wenz CLA 2010-12-16 09:26:35 EST
Of course M5 not M4
Comment 2 Michael Wenz CLA 2011-01-14 03:30:59 EST
I changed the JavaDoc for DefaultContextButtonPadData so that it allows subclassing and added a word of warning regarding changing default behavior that might lead to inconsistent behavior accross different Graphiti editors.

Michael
Comment 3 Michael Wenz CLA 2011-06-24 09:07:54 EDT
Part of Graphiti Indigo 0.8.0