-
Notifications
You must be signed in to change notification settings - Fork 184
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
UI - pan/zoom enhancements #126
Comments
Currently there already is "Save Graph" button on top-left. A/ add buttons in row with "Save Graph"
|
Actually - having the "show relations" mode active by default and on mouse over is pretty annoying. Could we change to
|
Hey @vgheo, your proposals seems good! I think having different modes (panning, selecting, zooming) as well as reseting the view will improve the usability. I currently don't have the bandwidth to implement this right now, but if you open up a PR with this implementation, I'll be more than happy to review and merge it 😄 |
@im2nguyen - I want to contribute this change - just want to make sure that you agree with the functionality |
PROPOSAL - v2 Panning modeMouse drag (with no key pressed) - panning Highlight and select mode( Currently this mode is active by default) Hold Ctrl
Zoom by selection
Reset viewButton: "Reset view"
NOTE: New buttons - added in line with existing "Save Graph" |
rover is using vue-cytoscape integration for the graph rendering Components API | vue-cytoscape pan/zoom logic is likely to be configured on the component https://github.com/im2nguyen/rover/blob/main/ui/src/components/Graph/Graph.vue#L399 |
Currently
Issue is that if we have a large module and a close zoom, it cannot be panned as there's no background in the current view.
PROPOSALS
Panning mode
Hold + mouse drag - always zooms
Alternative: introduce a toolbox ( Pan Mode / Select Mode )
Zoom by selection
Reset view
Button: "Reset view"
NOTE: New buttons - added in line with existing "Save Graph"
@im2nguyen - please tell us your thoughts
The text was updated successfully, but these errors were encountered: