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

GHCJS.VDOM.Unsafe: exporting js_vnode and internal javascript references... #7

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

Conversation

wavewave
Copy link
Member

@wavewave wavewave commented Apr 5, 2015

... for Properties, Children, Patch and VNode

I exported those to make use of ghcjs-vdom for webcomponent custom element (such as Polymer elements). This issue was discussed in #6 and #3.

…ces for Properties, Children, Patch and VNode
@cliffordbeshers
Copy link

Seems like a good alternative to my patch.

@arianvp
Copy link
Contributor

arianvp commented Apr 7, 2015

Your idea here is that the Unsafe module is for custom stuff and the standard module only exports smart constructors for known elements? I like the idea. I'm OK with merging. @luite thoughts?

@wavewave
Copy link
Member Author

wavewave commented Apr 7, 2015

Yes. In addition, it is often needed to use unProperties in Properties to do some exotic stuff. With this patch, I successfully use polymer element in my web app in conjuction with ghcjs-dom-delegator. I will make some example code for this some time soon.

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.

3 participants