Skip to content

Fix zooming#1347

Merged
SylvainCorlay merged 2 commits intobqplot:masterfrom
maartenbreddels:fix_zoom_lab3
May 11, 2021
Merged

Fix zooming#1347
SylvainCorlay merged 2 commits intobqplot:masterfrom
maartenbreddels:fix_zoom_lab3

Conversation

@maartenbreddels
Copy link
Copy Markdown
Member

@maartenbreddels maartenbreddels commented May 11, 2021

This fixes #1304 and fixes #81 (Whow, that's an oldie!)

@maartenbreddels
Copy link
Copy Markdown
Member Author

This cleans up all the event handlers and uses the now
standard 'wheel' event, which has as by effect that
the page does not scroll anymore with firefox.
@pllim

This comment has been minimized.

@SylvainCorlay
Copy link
Copy Markdown
Member

yay!

@SylvainCorlay SylvainCorlay merged commit 860dd69 into bqplot:master May 11, 2021
@pllim
Copy link
Copy Markdown
Contributor

pllim commented May 11, 2021

I wanted to test this but unable to install from source. I use conda to install nodejs and I don't see any upgrade available.

error human-signals@2.1.0: The engine "node" is incompatible with this module. Expected version ">=10.17.0". Got "10.13.0"
    error Found incompatible module.

I guess I'll just have to trust you and wait for a release. Thanks!

@maartenbreddels
Copy link
Copy Markdown
Member Author

maartenbreddels commented May 11, 2021 via email

@pllim
Copy link
Copy Markdown
Contributor

pllim commented May 21, 2021

Instead of messing with nodejs, I got something to install by hacking your yarn.lock before installing from source:

diff --git a/js/yarn.lock b/js/yarn.lock
index ace7b2fb..c7f003d3 100644
--- a/js/yarn.lock
+++ b/js/yarn.lock
@@ -3756,7 +3756,6 @@ execa@^5.0.0:
   dependencies:
     cross-spawn "^7.0.3"
     get-stream "^6.0.0"
-    human-signals "^2.1.0"
     is-stream "^2.0.0"
     merge-stream "^2.0.0"
     npm-run-path "^4.0.1"
Building wheels for collected packages: bqplot
  Building wheel for bqplot (PEP 517) ... done
  Created wheel for bqplot: filename=bqplot-0.12.26-py2.py3-none-any.whl ...
  Stored in directory: ...
Successfully built bqplot
Installing collected packages: bqplot
Successfully installed bqplot-0.12.26

Not sure why the version isn't reporting "dev" and also zoom still does not work for me.

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.

Zoom is not working anymore Scroll behavior of notebook while zooming in firefox

3 participants