Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Specify sides from which to connect a node #19

Closed
monomon opened this issue Apr 9, 2013 · 5 comments
Closed

Specify sides from which to connect a node #19

monomon opened this issue Apr 9, 2013 · 5 comments

Comments

@monomon
Copy link
Contributor

monomon commented Apr 9, 2013

When creating node/edge specify sides on which to allow connections
(e.g. allow only north and south)
This is a bit specific, but it might be useful. It is a small change in graffle, will post a patch soon.
Let me know if you think it fits, though.

@strathausen
Copy link
Owner

You still have that patch? If it's small, I wouldn't mind... but the code base has changed a lot since then. Closing for now but feel free to re-open :-)

@monomon
Copy link
Contributor Author

monomon commented Oct 11, 2017

I do, in fact! Let me bring the code up to date and see if it works.
I ended up implementing that in a separate "overrides" file, which adds some functions on the Graph and Raphael prototypes, so I'll take some time to integrate it better and shape it up.
Edit: I'm attaching a screenshot of how it works visually.

vtkkonnekt

@monomon
Copy link
Contributor Author

monomon commented Oct 13, 2017

OK the required changes were more than I reckoned, but I might as well do it. Will just take a bit longer, but I get the chance to build my code around some new js fanciness.

@monomon
Copy link
Contributor Author

monomon commented Nov 4, 2017

Hey, I am still implementing the changes, but my test code is not working, so I'm trying to find out whether I'm missing something changed e.g. in layouts. Just wanted to let you know it's in progress.

@monomon
Copy link
Contributor Author

monomon commented Nov 16, 2017

I have created a pull request for this change #45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants