Skip to content
This repository has been archived by the owner on Jun 16, 2020. It is now read-only.

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

Open
mpal9000 opened this issue Aug 8, 2015 · 2 comments
Open

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

mpal9000 opened this issue Aug 8, 2015 · 2 comments

Comments

@mpal9000
Copy link

mpal9000 commented Aug 8, 2015

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

@mpal9000
Copy link
Author

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
Copy link
Author

mpal9000 commented Sep 3, 2015

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant