search

ogallagher / fxgraph

A JavaFX graph editor
Do What The F*ck You Want To Public License
2 stars 0 forks source link

SequenceDiagram.layout functionally overrides Graph.layout #8

Open ogallagher opened 2 years ago

ogallagher commented 2 years ago

The way that SequenceDiagram extends Graph overrides Graph.layout implicitly by using a slightly different method signature is confusing, so I propose making the override explicit by having the layout parameter be optional.