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

Posts are SAVE instead of SAVED if you filter the Reading List #1785

Closed
ItsASine opened this issue Feb 12, 2019 · 6 comments
Closed

Posts are SAVE instead of SAVED if you filter the Reading List #1785

ItsASine opened this issue Feb 12, 2019 · 6 comments
Assignees
Labels
bug always open for contribution external contributors welcome contribution is welcome!

Comments

@ItsASine
Copy link

Describe the bug
I know there are other issues around SAVE/SAVED but this one seems unique enough for its own post. When filtering the Reading List, the buttons on posts are not in the SAVED state.

To reproduce:

  • Navigate to the Reading List
  • Select one of the tags on the sidebar

Expected:

  • Articles are marked as Saved as they are in the list of saved stuff

Actual:

  • Article button is Save as if they are not in the Reading List

Screenshots
image

Desktop (please complete the following information):

  • OS: Chrome OS
  • Browser: Chrome
  • Version: 71.0.3578.127
@jessleenyc jessleenyc added the bug always open for contribution label Feb 12, 2019
@jessleenyc
Copy link
Contributor

Hmm yeah, this seems related with #1395.

@jessleenyc jessleenyc added the external contributors welcome contribution is welcome! label Feb 12, 2019
@jessleenyc
Copy link
Contributor

@lightalloy I know you've been looking at reading list so wanted to get this on your radar too.

@ItsASine
Copy link
Author

ItsASine commented Feb 13, 2019

@jessleenyc I almost logged this as a comment on #1395 but that seemed more focused on the initial state being SAVE and not always toggling to SAVED, meanwhile, this is every time you try filtering the list, the button is in the wrong state for each article. Could be an issue with the initial state, but seemed more keen on being an issue with the filtering.

@lightalloy
Copy link
Contributor

@jessleenyc I suppose it's related to the frontend side (but needs research). I'll look at this after the other tickets.

@lightalloy
Copy link
Contributor

Currently working on this one. I can confirm that it's related to #1395 . On page load, the code which determines the button states runs several times and sometimes finishes with the wrong button state.
On clicking the tag (as described in this issue) the code just doesn't run and all buttons remain with the "SAVE" label.

@lightalloy
Copy link
Contributor

Closed by #1927

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug always open for contribution external contributors welcome contribution is welcome!
Projects
None yet
Development

No branches or pull requests

3 participants