Skip to content

Conversation

@noahmay
Copy link
Contributor

@noahmay noahmay commented Aug 3, 2022

Zoom in and out buttons are now disabled when reach max/min zoom. This fixes and issue when the zoom to fit sets a lower zoom scale than min zoom scale which made the zoom out button zoom in to min zoom scale. Also updates the min zoom scale on zoom to fit.

@OskarDamkjaer OskarDamkjaer self-requested a review August 3, 2022 13:03
Copy link
Contributor

@OskarDamkjaer OskarDamkjaer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge on green

@noahmay noahmay merged commit 171812f into neo4j:master Aug 4, 2022
@noahmay noahmay deleted the zoom-out-behaviour-with-fit-to-zoom branch August 4, 2022 11:04
noahmay added a commit to noahmay/neo4j-browser that referenced this pull request Aug 4, 2022
Disables zoom in and out buttons when reaching min/max zoom.
noahmay added a commit that referenced this pull request Aug 4, 2022
* Visualization zooms to cusor

* Prevent 5.0 flakyness from async db creation (#1786)

* Add util fn for properly creating databases to avoid flakyness from slow database creations

* Run run multidb tests only on dbs with full SHOW DATABASE command

* Get rid of jfrog (#1788)

* Fix bug with incorrect version number & add git hash to overview (#1791)

* Fix bug with incorrect version number & add git hash to overview

* Inject instead of fetching

* Remove dynamic import (#1787)

* Remove dynamic import

* Help typescript along

* update tests

* Adds background for usage of wheelDelta

* Update viz.spec.ts

* Include data cleanup in test

* Zoom out behaviour with fit to zoom (#1790)

Disables zoom in and out buttons when reaching min/max zoom.

* remove it.only from test case

Co-authored-by: Oskar Damkjaer <[email protected]>
Co-authored-by: Oskar Damkjaer <[email protected]>
noahmay added a commit to OskarDamkjaer/neo4j-browser that referenced this pull request Aug 22, 2022
Disables zoom in and out buttons when reaching min/max zoom.
noahmay added a commit to OskarDamkjaer/neo4j-browser that referenced this pull request Aug 22, 2022
* Visualization zooms to cusor

* Prevent 5.0 flakyness from async db creation (neo4j#1786)

* Add util fn for properly creating databases to avoid flakyness from slow database creations

* Run run multidb tests only on dbs with full SHOW DATABASE command

* Get rid of jfrog (neo4j#1788)

* Fix bug with incorrect version number & add git hash to overview (neo4j#1791)

* Fix bug with incorrect version number & add git hash to overview

* Inject instead of fetching

* Remove dynamic import (neo4j#1787)

* Remove dynamic import

* Help typescript along

* update tests

* Adds background for usage of wheelDelta

* Update viz.spec.ts

* Include data cleanup in test

* Zoom out behaviour with fit to zoom (neo4j#1790)

Disables zoom in and out buttons when reaching min/max zoom.

* remove it.only from test case

Co-authored-by: Oskar Damkjaer <[email protected]>
Co-authored-by: Oskar Damkjaer <[email protected]>
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 this pull request may close these issues.

2 participants