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

Allow zooming by node name in a single graph #12

Closed
EatingW opened this issue Jul 6, 2020 · 0 comments · Fixed by #23
Closed

Allow zooming by node name in a single graph #12

EatingW opened this issue Jul 6, 2020 · 0 comments · Fixed by #23
Assignees

Comments

@EatingW
Copy link
Collaborator

EatingW commented Jul 6, 2020

Poker planning notes:

  • Goals of the task:
    • Adding an input field for the user to search the name of the node and have the graph zoom in on that node
    • use case: user can inspect one node and surrounding nodes.
    • The underlying graph wouldn't change, the zoom only affect what the users see.
    • will only zoom in on the node if it exists
    • if the node doesn't exist, display this to the user
  • How to control the zoom?
    • cytoscape can specify a zoom
  • Graph is stored in the server. Graph is passed via json to the client. Can do everything in the client
  • How to zoom out?
    • A separate button? Empty the text box?
  • Steps:
    • Text field to get input from the user
    • Investigate API for cytoscape to see whether it allows zooming (there's an example on their website http://www.wineandcheesemap.com/)
    • On the client display the zoomed in graph
  • How to consider this done?
    • unit tests for the search functionality + return error if not exist
    • manually verify that the zoom functionality works

Proposed estimate: 2

@connorbrett connorbrett self-assigned this Jul 7, 2020
@EatingW EatingW changed the title Allow filtering by node name in a single graph Allow zooming by node name in a single graph Jul 16, 2020
@connorbrett connorbrett linked a pull request Jul 16, 2020 that will close this issue
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 a pull request may close this issue.

3 participants