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

add perfect-scrollbar as dependency #96

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mischkl
Copy link

@mischkl mischkl commented Feb 8, 2018

Fix #95

@isherwood
Copy link
Collaborator

isherwood commented Feb 9, 2018

Thank you. This looks good. Should the Code page docs (README.md) be updated to match? NPM could be added as an install option.

@mischkl
Copy link
Author

mischkl commented Feb 12, 2018

Good idea. I've now updated the README in my pull request to include information about installing with NPM and using with Webpack.

@Maximaximum
Copy link

Are there any ideas about when is this going to be merged?

@isherwood
Copy link
Collaborator

isherwood commented May 10, 2018

Has anyone tested it? It's not difficult to clone the repo and have a look at how the NPM install behaves. It's been fairly low on my list since I don't develop in AngularJS anymore (having moved to Angular5).

If we get some confirmation that it looks good I'll merge it.

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.

3 participants