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

Fix: Call Element.newElement as a native function #207

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@ThomasWeiser
Contributor

ThomasWeiser commented Apr 4, 2015

newElement is now a native function in Native.Graphics.Element.js

It is not exported by Element.elm.
So calling it as Element.newElement throws a runtime error

Call Element.newElement as a native function
newElement is now a native function in Native.Graphics.Element.js

It is not exported by Element.elm.
So calling it as Element.newElement throws a runtime error
@ThomasWeiser

This comment has been minimized.

Show comment
Hide comment
@ThomasWeiser

ThomasWeiser Apr 18, 2015

Contributor

It's fixed in the meantime

Contributor

ThomasWeiser commented Apr 18, 2015

It's fixed in the meantime

@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Apr 19, 2015

Member

Wait, so this got fixed since it was opened?

Member

evancz commented Apr 19, 2015

Wait, so this got fixed since it was opened?

@ThomasWeiser

This comment has been minimized.

Show comment
Hide comment
@ThomasWeiser

ThomasWeiser Apr 20, 2015

Contributor

Yes, it got fixed with your commit 5b0bc59

Contributor

ThomasWeiser commented Apr 20, 2015

Yes, it got fixed with your commit 5b0bc59

@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Apr 20, 2015

Member

Cool, sorry for the confusion and thanks for the PR!

Member

evancz commented Apr 20, 2015

Cool, sorry for the confusion and thanks for the PR!

@ThomasWeiser ThomasWeiser deleted the ThomasWeiser:fix-newElement branch May 25, 2015

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