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

Added events to elm version #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

flip111
Copy link
Contributor

@flip111 flip111 commented Feb 22, 2017

added events in the elm code, unfortunately for the window.onClick i had to use ports
https://stackoverflow.com/questions/39652083/how-can-i-add-event-handlers-to-the-body-element-in-elm

which breaks elm-reactor
elm-lang/elm-reactor#36

now the javascript file should be created with
elm-make main.elm --output main.js

@flip111 flip111 force-pushed the master branch 2 times, most recently from 9f93e82 to dfb027c Compare February 22, 2017 23:06
@flip111
Copy link
Contributor Author

flip111 commented Feb 22, 2017

If you want to put main.js through the google closure compiler then use --assume_function_wrapper while building and put closure compiler in SIMPLE mode (there are issues with ADVANCED mode)

java -jar closure-compiler.jar -O SIMPLE --assume_function_wrapper --js main.js > /tmp/main.js && mv /tmp/main.js main.js

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.

1 participant