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

Dashboard: Status filter + author filter not working well together #11668

Closed
spacedmonkey opened this issue Jun 10, 2022 · 5 comments · Fixed by #11851
Closed

Dashboard: Status filter + author filter not working well together #11668

spacedmonkey opened this issue Jun 10, 2022 · 5 comments · Fixed by #11851
Labels
Package: Dashboard /packages/dashboard Type: Bug Something isn't working

Comments

@spacedmonkey
Copy link
Contributor

Bug Description

If you have a story status filter set and then you filter by author, if that author does not have a anything in that status, then story status should reset to all or something.

See video.

Expected Behaviour

Steps to Reproduce

  1. Open dashboard.
  2. Filter by pending.
  3. Filter by author ( author must not have pending story ).
  4. See status filter not highlighted.

Screenshots

Screen.Recording.2022-06-10.at.11.37.09.mov

Additional Context

  • Plugin Version: 1.22.0
  • WordPress Version: 6.0
  • Operating System:
  • Browser:
@sblinde
Copy link
Contributor

sblinde commented Jun 10, 2022

Great find! We added this to the epic for filtering improvements / enhancements.

@sblinde
Copy link
Contributor

sblinde commented Jun 23, 2022

Linking this reported issue to #11789, and removing estimate as this will be an incident report that should be resolved with that greater work.

@mwritter
Copy link
Contributor

mwritter commented Jun 28, 2022

@spacedmonkey @swissspidy @sblinde - Im currently addressing this issue in #11789 and Im working on implementing a solution but wanted to get some feedback on how to handle this issue.

if that author does not have anything in that status, then story status should reset to all or something.

🤔 The 'status filter' updates every time the user changes any other filter, so once the other filter updates we make a call to the backend for the stories, which gives us the 'status counts' for the given set of filters. So once we found that there are no stories for the given set of filters we would have to make a second call to the backend to 'reset' the status filter. Im not sure if we want to do that. Any insight on a better way to 'reset' would be greatly appreciated if we do wanna go down that path.

I do have a working solution that gives a better visual cue to the user though to let them know they're filtering by a status that has no results, any feedback on this implementation or suggestion would be great.

Screen.Recording.2022-06-28.at.11.17.34.AM.mov

@swissspidy
Copy link
Collaborator

What you demo in the video seems fine to me. I wouldn't further complicate it though by doing multiple requests or anything.

@swissspidy swissspidy changed the title Status filter + author filter not working well together. Dashboard: Status filter + author filter not working well together Jun 29, 2022
@sblinde
Copy link
Contributor

sblinde commented Jun 30, 2022

Agreed w/ @swissspidy as well!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: Dashboard /packages/dashboard Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants