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
AMD compatibility & manual initialization #15
base: master
Are you sure you want to change the base?
Conversation
Any update on this? Can you please take a look @bjork24? |
Btw, this pull request made the difference for me. I ended up doing to fire stuff on load: Unison.util.initializeUnison();
// ... add event listeners
Unison.emit(Unison.fetch.now().name); |
It seems as if @bjork24 is not really maintaining this anymore. But I guess we can live with forks. |
Adding "off" method for unsubscribing to events
node_modules | ||
.DS_Store |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since .DS_Store
is an environmental file, rather than a project file, you might want to add that into your global gitignore config instead.
Hi
I have added some changes in my custom branch to support AMD loaders and to be able to initialize Unison "manually". The problem with the original implementation was, that if included it inside your own
DOMContentLoaded
listener, the event inside Unison never fires (since it fires once). I added the publicutil.initializeUnison
method which can be called manually and initializes Unison.Also, I added an
.editorconfig
file to support your syntax and made some format changes. Feel free to include this in your version.P.S. Unison should still be exposed to the global namespace, so I assume that your comments plugin still works. However, I wasn't able to test this. Also, I have not added a minified version since I wasn't sure which minifier you usually use for your releases.