Results 18 comments of Carlo Bernaschina

Extra info: To reproduce it is enough to build a model with `100000` nodes connected in a line `n0 => n1 => n2 => .... => n100000`.

Thank you very much for these fixes. Is there an idea of when these changes will get into a release?

Just to fully understand. The dev release you are suggesting would be for cherry picking the changes into `elkjs` and have a point release there?

Friendly Ping. Will you have time to `assemble a dev release based on the current master for elkjs`?

Version 0.7.3-dev solved the problem with `$connectedComponentsDFS`, but it is triggering it again in `$tightTreeDFS`. https://github.com/eclipse/elk/blob/3630e23fa5abc253233296e2bdf5e18828e5dba7/plugins/org.eclipse.elk.alg.common/src/org/eclipse/elk/alg/common/networksimplex/NetworkSimplex.java#L515-L537 Which is also recursive

I tried to change the strategy to any other value, e.g.: ```javascript layoutOptions: { // ... 'elk.layered.nodePlacement.strategy': 'LINEAR_SEGMENTS', } ``` But nothing changes. (let me know if I'm doing it...

In the end I added a regular expression that fixes this and other problems. ![image](https://cloud.githubusercontent.com/assets/3967880/13374379/7cb1151a-dd83-11e5-9627-c275e8cc2acb.png) I am not sure here. It's necessary to use no-break spaces otherwise IE would collapse...

And Chrome is sending a warning about the method used for the animation. > SVG's SMIL animations (, , etc.) are deprecated and will be removed. Please use CSS animations...

Note that a lot of this code could be simplified by using `std::min` and `std::max`. `std::clamp` could also be used in various places, but that would be limited to C++17...