search

ArthurClemens / mithril-slider

Content slider for Mithril for mobile and desktop
30 stars 5 forks source link

Do not modify vnode.state directly, fixes #7 #8

Closed benmerckx closed 6 years ago

benmerckx commented 6 years ago

This PR avoids assigning vnode.state directly, for reasons described in #7. Not sure if this is the way to go about it though :)

ArthurClemens commented 6 years ago

Thanks!

benmerckx commented 6 years ago

Any chance for a release? As the current npm release is not compatible with newer mithril version.

ArthurClemens commented 6 years ago

Thanks for reminding. I had forgotten about this.

ArthurClemens commented 6 years ago

Done