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

Filtered List Bug (Found in the Todo-List Example) #2240

Closed
phas0r opened this issue Feb 1, 2017 · 2 comments
Closed

Filtered List Bug (Found in the Todo-List Example) #2240

phas0r opened this issue Feb 1, 2017 · 2 comments
Assignees
Labels

Comments

@phas0r
Copy link

@phas0r phas0r commented Feb 1, 2017

  1. Describe your issue:
    Clicking the 'filter'-button does not always filter the correct items.

  2. Can you reproduce the issue?
    http://riotjs.com/examples/plunker/?app=todo-app
    or
    Alternative saved Plunk
    Check the second item so that the first and the second item are marked as done and the third item is not marked as done. Then click the filter button.
    It removes the first item and the third item and leaves the second one, rendered as done.
    It seems to be related to the second item in the list (you can remove the hidden one and the problem still exists).
    If you have four items and mark the first three then it removes 1, 3 and 4.
    If you have four items and mark the first two then it removes 1 and 3.
    It seems to always remove the next following unmarked item in the list and leave the second one rendered as marked.

  3. On which browser/OS does the issue appear?
    Chrome Version 55.0.2883.87 m
    Windows 7

  4. Which version of Riot does it affect?
    The example seems to use the latest version in the master branch.

  5. How would you tag this issue?

  • Question
  • Bug
  • Discussion
  • Feature request
  • Tip
  • Enhancement
  • Performance
@phas0r phas0r changed the title Bug in the Todo-List Example Filtered List Bug (Found in the Todo-List Example) Feb 1, 2017
@GianlucaGuarini GianlucaGuarini added the bug label Feb 2, 2017
@GianlucaGuarini
Copy link
Member

@GianlucaGuarini GianlucaGuarini commented Feb 2, 2017

could you please move this issue here https://github.com/riot/examples?

@GianlucaGuarini
Copy link
Member

@GianlucaGuarini GianlucaGuarini commented Feb 2, 2017

nevermind leave it here, I want to check whether the issue is in the example or in riot

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.