search

Raynos / mercury

A truly modular frontend framework
http://raynos.github.io/mercury/
MIT License
2.82k stars 143 forks source link

`hg.array()` - remove from core comment #177

Open mpal9000 opened 9 years ago

mpal9000 commented 9 years ago

https://github.com/Raynos/mercury/blob/master/index.js#L46 Hello is the above comment correct?

mpal9000 commented 9 years ago

I can now understand that cryptic comment and I agree that observ-array should be removed or fixed. I hope it will be fixed, because it's not nice to use observ-varhash for arrays.

I had an array of observables and was trying to render them. The render function was using an ev-click: hg.sendClick(state.channes.onClick) property and I was also using "./time-travel.js". Without time-travel it was working as it should, but after an undo the click handlers was triggered, but the state updates did not reflect to the UI.

mpal9000 commented 9 years ago

This describes the problem better: https://github.com/mpal9000/mercury-example-1 (see comments inside ./index.js)