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

Highlight both incoming and outgoing links when hovering over a node #2

Open
jdiamond opened this issue Apr 2, 2013 · 4 comments
Open

Comments

@jdiamond
Copy link

jdiamond commented Apr 2, 2013

I think it would be useful if dependo could highlight both incoming and outgoing links when hovering over a node. Right now it only highlights the outgoing links.

Maybe holding down different modifier keys (alt, ctrl) while hovering could hide one or the other sets.

@auchenberg
Copy link
Owner

Good idea. I actually had all the nodes highlighted to begin with, but has limited to only take direct children, after feedback some others.

How about adding some controls, so you can toggle how it should work?

@jdiamond
Copy link
Author

jdiamond commented Apr 3, 2013

Yeah, I'd be happy with controls or even just a variable that I can set in the JavaScript console. =)

I tried making some changes in the JavaScript embedded in my report but only got it to half work before opening this issue.

I changed this:

    // Build linked index
    data.links.forEach(function(d) {
      linkedByIndex[d.source.index + "," + d.target.index] = 1;
    });

to this:

    // Build linked index
    data.links.forEach(function(d) {
      linkedByIndex[d.source + "," + d.target] = 1;
    });

That fixed it so that the incoming nodes aren't dimmed. I couldn't figure out how to not dim the links.

@npradeep
Copy link

Hi ,

I have implemented in one of my page but this is not highlighted even out going links also. I found the issue, with the name having special characters and spaces. So if any name with special characters and spaces it never highlighted the outgoing links. Can anyone help on this issue.

@christianacca
Copy link

+100 for being able to add controls to be able to see incoming and/or outgoing dependencies :-)

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

No branches or pull requests

4 participants