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

WIP change cursor to hand cursor when hovering over the tree and grabbing cursor when dragging the tree #129

Closed
wants to merge 1 commit into from

Conversation

kellydoleo
Copy link
Contributor

I've managed to update the cursor to a hand/grab cursor when hovering over the tree on the edam browser. This would allow users to know they can drag the tree. However, I'm still attempting to have the cursor change when dragging the tree @bryan-brancotte if you have any recommendations on where I can look more into this, I would greatly appreciate any help!

@kellydoleo
Copy link
Contributor Author

I'm reviewing the documentation on d3 drag in order to get the cursor to change to the grabbing cursor when dragging across the tree. It mentioned that I would need d3-drag npm package, so those files are also included in my PR.

@bryan-brancotte
Copy link
Member

Hi @kellydoleo, quick tip: node_modules should never be push to a repo, to prevent futur commit I added it to .gitignore

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

Successfully merging this pull request may close these issues.

2 participants