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

Each and switching between object and array #2027

Closed
pavel opened this issue Oct 16, 2016 · 2 comments
Closed

Each and switching between object and array #2027

pavel opened this issue Oct 16, 2016 · 2 comments
Labels
Milestone

Comments

@pavel
Copy link

@pavel pavel commented Oct 16, 2016

  1. Describe your issue:
    each={ item in iterable } stops working when the iterable is switched from an object to an array.
  2. Can you reproduce the issue?
    Demo URL: http://jsfiddle.net/k6L77wpx/3/.
    To reproduce:
    • click "Switch" one time (switch from array to object). Notice that data changes.
    • Hit "Switch" second time (switch from object to array). Notice how this.current changes while each does not reflect the change.
    • From this point each does not reflect changes anymore.
  3. On which browser/OS does the issue appear?
    Any browser/OS.
  4. Which version of Riot does it affect?
    Tested against riot@next and v3.0.0-alpha.11. Testing against previous versions makes little sense due to #1420.
  5. How would you tag this issue?
    • Question
    • Bug
    • Discussion
    • Feature request
    • Tip
    • Enhancement
    • Performance
@GianlucaGuarini GianlucaGuarini added the bug label Oct 16, 2016
@GianlucaGuarini
Copy link
Member

@GianlucaGuarini GianlucaGuarini commented Oct 16, 2016

@pavel nice one thanks! This seems to be a bug but I need to verify what's wrong

@pavel
Copy link
Author

@pavel pavel commented Oct 16, 2016

@GianlucaGuarini You're welcome.

danielkrainas added a commit to danielkrainas/riot that referenced this issue Oct 18, 2016
@GianlucaGuarini GianlucaGuarini added this to the 3.0.0 milestone Oct 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.