search

tesis-dynaware / graph-editor

Eclipse Public License 1.0
132 stars 42 forks source link

Make "GraphEditorContainer.getChildren()" public #23

Closed eckig closed 9 years ago

eckig commented 9 years ago

As the title reads.. please increase the visibility of GraphEditorContainer.getChildren() to public.

We need this modifiable List of children to add additional overlays similar to the already existing minimap.

Yes, we could add/remove these overlays on a different Node above (additional layer) but this increases complexity and makes it hard to manage different overlays.

rmfisher commented 9 years ago

Sounds reasonable. I exposed it but haven't tested adding other children yet so no guarantees it will work properly :)

eckig commented 9 years ago

Alread did this, with reflection (which is always kind of ugly) and it works just fine. So thanks for adding a proper way of doing this.