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 zoom on dark matter graph #1

Closed
kadrlica opened this issue Dec 29, 2017 · 5 comments · Fixed by #17
Closed

Allow zoom on dark matter graph #1

kadrlica opened this issue Dec 29, 2017 · 5 comments · Fixed by #17

Comments

@kadrlica
Copy link
Contributor

Transferring from LSSTDESC/LSSTDarkMatter#9 ...

Is it possible to allow the user to dynamically change the zoom of the dark matter graph? I'm able to change this with the browsers zoom level, but it might be useful to make this part of the webapp. (Also, I expect that the content will only grow, and there may be some reason someone would want to visualize the whole network at once.)

@yymao
Copy link
Member

yymao commented Dec 31, 2017

@kadrlica now that I think about it, I wonder if there's really a need to reimplement the zoom feature given that the zoom feature is a standard in all modern browsers.

Do you just want a "fit to window" button for this purpose?

@kadrlica
Copy link
Contributor Author

I don't know how difficult this is, so I'll defer to your experience. My main thought was to have some visual cue that zooming was possible, even if it just used the built in browser functionality (this may be impossible).

This issue can probably wait until we start seeing the full complexity of the network graphic. I think that if we start to have 50+ measurements then this issue may resurface.

@yymao
Copy link
Member

yymao commented Dec 31, 2017

In that case, we can simply add text to instruct the user to use the browser built-in zoom function.

@kadrlica
Copy link
Contributor Author

Yes, I think that would be fine for now.

@yymao
Copy link
Member

yymao commented Dec 31, 2017

As this thing expands, we probably need a "front page" to example what we are doing and provide some instructions to the functionalities of each of these representations.

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

Successfully merging a pull request may close this issue.

2 participants