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.

Sign up for GitHub

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

Jump to bottom

Calling .remove() throws an error after update to v. 3.7.4 #681

Closed
sebhewelt opened this issue Jul 8, 2017 · 1 comment
Closed

Calling .remove() throws an error after update to v. 3.7.4 #681

sebhewelt opened this issue Jul 8, 2017 · 1 comment
Labels
on next release

Comments

@sebhewelt
Copy link

In my react app I call .remove() method on body:

 componentWillUnmount() {
   
 $("body").getNiceScroll().remove();

  }

It throws an error:

Uncaught ReferenceError: del is not defined
at NiceScrollClass.undelegate

Is remove() deprecated? How to remove the scrollbar completely other way?

cheers.
@inuyaksa
Copy link
Owner

Fix soon

inuyaksa added a commit that referenced this issue Jul 10, 2017
@inuyaksa
3.7.5-b [DEV]
41d2c38 
Fixes
- Calling .remove() throws an error after update to v. 3.7.4 #681
- more robust hybrid system support mouse + touch events aka no drag and
scroll #676

Changes
- remove outdated self.visibility
inuyaksa added a commit that referenced this issue Jul 10, 2017
@inuyaksa
3.7.5-c [DEV]
3232047 
Fixes
- Calling .remove() throws an error after update to v. 3.7.4 #681
- more robust hybrid system support mouse + touch events aka no drag and
scroll #676
- typos on navigator.maxTouchPoints

Changes
- remove outdated self.visibility
inuyaksa added a commit that referenced this issue Jul 13, 2017
@inuyaksa
3.7.5 - fix for touchscreen desktops
708dc57 
Fixes
- Calling .remove() throws an error after update to v. 3.7.4 #681
- more robust hybrid system support mouse + touch events aka no drag and
scroll #676
- scroll is not working on edge and firefox browsers in touchscreen
laptops #677
- typos on navigator.maxTouchPoints

Changes
- remove outdated self.visibility
inuyaksa added a commit that referenced this issue Jul 13, 2017
@inuyaksa
3.7.5 - fix for touchscreen desktops
94faa01 
Fixes
- Calling .remove() throws an error after update to v. 3.7.4 #681
- more robust hybrid system support mouse + touch events aka no drag and
scroll #676
- scroll is not working on edge and firefox browsers in touchscreen
laptops #677
- typos on navigator.maxTouchPoints

Changes
- remove outdated self.visibility
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
on next release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@inuyaksa @sebhewelt