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

clickToUse: mouseDown instead of mouseUp #41

Open
whme opened this issue Jul 29, 2019 · 1 comment
Open

clickToUse: mouseDown instead of mouseUp #41

whme opened this issue Jul 29, 2019 · 1 comment
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@whme
Copy link

whme commented Jul 29, 2019

Hey,
the clickToUse option is really helpfull and overall a nice feature. Especially if the graph is embedded in a page where you have to scroll so you can scroll through the page without accidently zooming in or out the graph.
However, if one wants to move around elements in the graph and has to select it first it just feels bad.
So instead of having the graph selected once the users click is done (so on the MouseButtonRelease or MouseButtonUp event) it would be nice to have it select already on the MouseButtonPress or MouseButtonDown event.
An option where you can switch between default and "new" behavior would maybe be the best.

@mojoaxel mojoaxel added enhancement New feature or request help wanted Extra attention is needed labels Jul 29, 2019
@Thomaash
Copy link
Member

This won't be that simple. Click to use is based on overlay that swallows events. The mouse down would be swallowed by the overlay anyway and another mouse down would be necessary to drag nodes around. This would require either synthetic events (these could be a bit tricky) or complete overhaul of click to use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants