Network: Zoom button doesn't work like expected #2100

Closed
nikitakurylev opened this Issue Sep 18, 2016 · 3 comments

Projects

None yet

3 participants

@nikitakurylev
Contributor
nikitakurylev commented Sep 18, 2016 edited

Hi,
Zoom button doesn't work correctly, it zooms to the center of network, not to the center of screen.
It supposed to work like traditional (google) map zoom button.
Not sure if it is bug.

Thanks and best regards

@tomzx
tomzx commented Sep 19, 2016 edited

Hello @nikitakurylev, could you provide a link to an example with those zoom buttons?

Using the mouse does zoom to where the cursor is, while zoom buttons would need a viewport coordinate to specify where the zoom should occur, none being given would imply the center of the viewport.

@nikitakurylev
Contributor

tos
Firstly I zoom it with mouse wheel, and then with zoom button.
I can get coordinates with getViewPosition(), but how can I set them for zoom button?

Thanks

@mojoaxel mojoaxel changed the title from Zoom button doesn't work correctly (Network) to Network: Zoom button doesn't work like expected Sep 20, 2016
@mojoaxel mojoaxel added enhancement and removed question labels Sep 20, 2016
@CapitanMorgan CapitanMorgan added a commit to StateFarmIns/vis that referenced this issue Oct 20, 2016
@CapitanMorgan CapitanMorgan Should fix issue #2100 and fix another issue we discovered around clu…
…sterByConnection leaving phantom edges.
db85c67
@CapitanMorgan CapitanMorgan added a commit to StateFarmIns/vis that referenced this issue Oct 24, 2016
@CapitanMorgan CapitanMorgan 2100 fix and cluster by connection fix (#1)
* Should fix issue #2100 and fix another issue we discovered around clusterByConnection leaving phantom edges. Also fixing a test that fails when you rebuild the project and run the tests.

* fixed a wrong variable name

* fixed DataView.test.js assert

* Update Canvas.js

* remove whitespace

remove end of file newline on Canvas.js

* strip end of file newlines
a4a32a7
@mojoaxel mojoaxel added this to the Minor Release v4.17 milestone Oct 24, 2016
@mojoaxel mojoaxel added a commit that referenced this issue Oct 25, 2016
@CapitanMorgan @mojoaxel CapitanMorgan + mojoaxel fixes #2100 and issues with edges when clustering (#2229)
* Should fix issue #2100 and fix another issue we discovered around clusterByConnection leaving phantom edges.
* fixed a wrong variable name
* fixed DataView.test.js assert
* Update Canvas.js
* remove whitespace; remove end of file newline on Canvas.js
* strip end of file newlines
* fixed styling. spacing, semicolons, line length
91c96ba
@mojoaxel mojoaxel closed this Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment