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

appear doesn't load in webpack #9

Closed
salvatorious opened this issue Sep 19, 2016 · 8 comments
Closed

appear doesn't load in webpack #9

salvatorious opened this issue Sep 19, 2016 · 8 comments

Comments

@salvatorious
Copy link

When requiring appear in a webpack-bundled file, it doesn't appear as a function.

Adding 'module.exports = appear;' to the end of appear.js fixes this problem, but I don't know if this would break the plugin for someone using it normally. Thoughts?

@ryanjadhav
Copy link

+1

1 similar comment
@intelligence
Copy link

+1

@salvatorious
Copy link
Author

Update: I bit the bullet and just got rid of appearjs.
https://www.npmjs.com/package/enter-view is fantastic so far, and the offset is from the bottom of the page, not the top, which is really nice.

@dzoba
Copy link
Contributor

dzoba commented Dec 9, 2016

+1

@JustinBeaudry
Copy link
Contributor

JustinBeaudry commented Jan 8, 2017

@salvatorious Thanks for the update! We don't use Webpack internally so it's great to get some feedback. We could add something like the following:

  if (typeof module !== 'undefined' && typeof module.exports !== 'undefined')
    module.exports = appear;
  else
    window.appear = appear;

PR's are welcome! 😄

@kurthoefer
Copy link

Hey any progress on this? I would love to use this with my react app!

@JustinBeaudry
Copy link
Contributor

@kurthoefer We're a bit swamped internally with other projects, but PR's are very welcome for this update.

JustinBeaudry added a commit that referenced this issue Jun 15, 2018
@JustinBeaudry
Copy link
Contributor

@salvatorious @kurthoefer Appear version 1.2.0 now supports webpack

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants