stache does not update view with can.List and deep properties #2007

Closed
pYr0x opened this Issue Oct 21, 2015 · 1 comment

Comments

Projects
None yet
3 participants
@pYr0x

pYr0x commented Oct 21, 2015

on checking out how to listen to viewModel-changes in components event object (#1740) i found a bug.

in this example http://jsbin.com/lakefonulo/edit?js,console,output the "mylist" item 0 is changed to "Hello" but the view is not updated.

@justinbmeyer justinbmeyer added the bug label Oct 29, 2015

@justinbmeyer justinbmeyer added this to the 2.3.2 milestone Oct 29, 2015

@justinbmeyer

This comment has been minimized.

Show comment
Hide comment
@justinbmeyer

justinbmeyer Oct 29, 2015

Contributor

This is almost certainly a problem with can.view.live.list not listening to set events.

Similar to how can.view.live.list listens to add events:

It needs to be doing a similar thing with set. But when a set events happens can sneakily call remove() and add methods with the updated item similar to how diffing does it: https://github.com/bitovi/canjs/blob/master/view/live/live.js#L402

Contributor

justinbmeyer commented Oct 29, 2015

This is almost certainly a problem with can.view.live.list not listening to set events.

Similar to how can.view.live.list listens to add events:

It needs to be doing a similar thing with set. But when a set events happens can sneakily call remove() and add methods with the updated item similar to how diffing does it: https://github.com/bitovi/canjs/blob/master/view/live/live.js#L402

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment