varkor / quiver

A modern commutative diagram editor for the web.
https://q.uiver.app
MIT License
2.43k stars 83 forks source link

Revisit offset heuristic #186

Closed varkor closed 1 year ago

varkor commented 1 year ago

There is currently a heuristic to work out how much new arrows should be offset, based on surrounding arrows. However, it is wrong as frequently as it is right. It would be good to rework this to make it more helpful.

varkor commented 1 year ago

One easy improvement would be to only consider edges with the same source and target, rather than same source or target.

varkor commented 1 year ago

This has been fixed by https://github.com/varkor/quiver/commit/6129410161b684b16480e2a2941582a3c218d460, using the suggested method in https://github.com/varkor/quiver/issues/186#issuecomment-1800131737.