Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 331290 - ContextButtonPadData extensibility
Summary: ContextButtonPadData extensibility
Status: CLOSED FIXED
Alias: None
Product: Graphiti
Classification: Modeling
Component: Core (show other bugs)
Version: 0.7.0   Edit
Hardware: All All
: P3 minor (vote)
Target Milestone: 0.8.0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard: Indigo M5 theme_round_offs
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-28 16:39 EST by Volker Wegert CLA
Modified: 2011-06-24 09:07 EDT (History)
2 users (show)

See Also:
michael.wenz: indigo+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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