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 support for specifying parent container #80

Closed
wants to merge 1 commit into from

Conversation

jasonLaster
Copy link

This updates #33, which has gone stale.

@gabegorelick
Copy link

+1

Any chance it will get merged this time?

@frankie-loves-jesus
Copy link

@gabegorelick hi! This is pretty much the same as the issue I just opened right? #84

@gabegorelick
Copy link

@frankie-loves-jesus Yes, it's a very common feature request.

@frankie-loves-jesus
Copy link

@jasonLaster could you merge https://github.com/jasonLaster/nprogress/tree/parent2 into your master so I can bower register nprogress-jasonlaster https://github.com/jasonLaster/nprogress/?

Also, I updated @jonjaques's testcase with his latest code:

http://jsbin.com/odOf/13/edit

@rstacruz
Copy link
Owner

Implemented. Thanks for the PR! See #33 and #86.

@frankie-loves-jesus
Copy link

Thank you @rstacruz!

@rstacruz
Copy link
Owner

Thank you guys too for reporting issues and sending out your PRs! If you'd like me to look at any specific issue that I may have overlooked, just ping me on Twitter/email/mention.

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.

5 participants