Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detect Map mutations #55

Open
cehoffman opened this issue May 19, 2016 · 4 comments
Open

Detect Map mutations #55

cehoffman opened this issue May 19, 2016 · 4 comments

Comments

@cehoffman
Copy link

Besides the problem in #54 when binding an empty Map it seems that any additions to a Map instance using map.set() are not detected. Are maps supported with virtual repeat? I thought that was part of abstract repeat strategy. The only way I've been able to get anything to show with a map is to go through a value converter that produces an array and even when I do that any changes in the map are not pushed through the value converter.

@martingust
Copy link
Contributor

@cehoffman The virtual-repeat correctly uses the same plugin approach as the standard repeat where different strategiest for iterating different collections can be plugged in. However the stratagies are quite different from the standarad repeat due to the virtualization aspect. And currently focus is to make the Array strategy solid. Then on to map, set etc. Basically a map implementation of virtual-repeat-array-strategy would need to be created and registrered here. But there is probably a lot of code that could be broken out and shared across the repos. If you want to helpout in implementing that it would be greatly appriciated!

@Alexander-Taran
Copy link
Contributor

stale since 2016

@Alexander-Taran
Copy link
Contributor

@bigopon not considered for au2?

@bigopon
Copy link
Member

bigopon commented Jun 3, 2022

it can be done for au2, just haven't started.

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

No branches or pull requests

4 participants