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

Destroy - Can actually create a slimScroller #87

Closed
robly opened this issue Aug 16, 2013 · 2 comments · Fixed by #144
Closed

Destroy - Can actually create a slimScroller #87

robly opened this issue Aug 16, 2013 · 2 comments · Fixed by #144

Comments

@robly
Copy link

robly commented Aug 16, 2013

You should place a check at line 152 in-case 'destroy' is passed in as an option so that it will not accidentally create a new slimScroller if one does not already exist.

Example: If you call slimScroll({destroy: true}); 2x it will create the scroller using the default values.

There might be a better place to put it, but that seems reasonable to me glancing at it.

@etetlow
Copy link

etetlow commented Oct 27, 2013

I'll second that. I wanted to use slimscroll on a responsive mobile solution for a client, but when going back to above the cutoff point of 1024px it was still showing alongside the desktop scrollbar. The desktop scrollbar was essential to an application used on the desktop view and had to find an alternative unfortunately.

@abhishekyadav9590
Copy link

its not solved yet

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 a pull request may close this issue.

3 participants