garyb / purescript-virtual-dom

Low level virtual-dom bindings for PureScript
MIT License
30 stars 7 forks source link

remove namespace and key parameters from vnode signature #5

Closed matthieubulte closed 9 years ago

matthieubulte commented 9 years ago

just a little cosmetic change to extract the namespace and the key of a vnode from the properties parameter instead of using two Maybe String.

It just makes the code look better and makes it lighter to use as most of the time these two parameters are not used.

garyb commented 9 years ago

Much preferrable, thanks. :+1: This is the approach virtual-dom/h takes too, which if I'd know about at the time I'd probably have gone with also.