Bug 242920 - Zest does not permit easy subclassing of GraphContainer to customize open/close methods
Summary: Zest does not permit easy subclassing of GraphContainer to customize open/clo...
Status: NEW
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy Zest (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows Mobile 2003
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: gef-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-01 16:06 EDT by Stefan Tucker CLA
Modified: 2015-01-22 02:44 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Tucker CLA 2008-08-01 16:06:26 EDT
Build ID: I20080617-2000

Steps To Reproduce:
I'd like to subclass GraphContainer and do different actions on open/close. I can override the GraphContainer's open() and close() methods, but I cannot easily set the container's ExpandGraphLabel (because it's in the zest.core.widgets.internal package, which isn't exported from the Zest core JAR).

It'd be great if the container handled opening and closing the ExpandGraphLabel and let a derived class override open() and close() to perform custom actions.

More information:
The open/close ActionEvents come from the embedded Clickable object, so the listener gets only that object. It's hard (if not impossible) to obtain from that the class derived from GraphContainer. So, I implement my class's own add-listener method which adds this object as the Clickable's listener. When it gets an event, it turns around and fires it at the original object that added my class as a listener. That doesn't seem very efficient.

Ideally, the event that comes from the GraphContainer will inform the listener of the node that was clicked and the state it's now in (open or closed).
Comment 1 Alexander Nyßen CLA 2015-01-22 02:44:55 EST
Re-assigning back to inbox, as Ian is no longer active committer.