eclipse / elk

Eclipse Layout Kernel - Automatic layout for Java applications.
https://www.eclipse.org/elk/
Other
258 stars 86 forks source link

Rectpacking does microlayout only after the layout step #996

Closed soerendomroes closed 9 months ago

soerendomroes commented 9 months ago

See here.

Since the micro layout executes after the layout step, the node size is only determined after everything is already placed (which is bad). We should consider doing it before the layout step.

soerendomroes commented 9 months ago

Something tells me that I had good reasons to put this at the end. Please investigate.

soerendomroes commented 9 months ago

The whitespace elimination wants it at the end. Maybe just do it at the beginning (for this bug) and at the end (for whitespace elimination?