search

mdeforall / atompm

A Tool for Multi-Paradigm Modeling
https://atompm.github.io/
GNU Lesser General Public License v3.0
0 stars 1 forks source link

A cell should be connected to already existing cells #8

Open hergin opened 3 years ago

hergin commented 3 years ago

Whenever a cell is dragged/created etc in a position, we should check if there are other existing cells around (immediate neighbors) and if there are, we should ensure that, there are links with each other.

For example, if there is an empty spot in between 8 cells like below:

XXX
X X
XXX

When we create another cell in between because of the snap guidelines, it should create the rest of the links as well. Not just the highlighted snap links.

hergin commented 3 years ago

The auto-link creation and snapping should be tested universally and designed better. There are still cases where it doesn't make sense.

Now recursive loop (#9) is solved but when the 8th one added like below, it doesn't make sense where it is created. Try and see.

To replicate, create the cells like below.

XXX
X X
X X

Then, add the 8th one just bottom-middle.

XXX
X X
XXX