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
feat: Timeline Filters #3284
feat: Timeline Filters #3284
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks pretty good 🚀
One bug i have found that if you deselect the filter Checkboxes quite quickly it cannot follow. (Image below)
The checkboxes show everything deselected but the badge shows only one filter, and that is also how the timelime behaves as events are still displayed. The severity of this might increase when the user uses a slower system/connection, or has a lot of timeline events.
Co-authored-by: Kuchenpirat <24235032+Kuchenpirat@users.noreply.github.com>
Co-authored-by: Kuchenpirat <24235032+Kuchenpirat@users.noreply.github.com>
Pretty sure the bug you saw with the checkbox being out of sync is fixed. Let me know if you're able to reproduce. I noticed it only got out of sync when clicking on the checkbox (vs clicking on the label) so try clicking on both, if you can |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks all good 👍
Can't reproduce the bug anymore. Neither by using the checkboxes, nor the labels themselves.
What type of PR is this?
(REQUIRED)
What this PR does / why we need it:
(REQUIRED)
Adds filters to the recipe timeline:
Filters are stored in user preferences so they're persisted. When one or more filters are selected, there's a badge on the button:
There are also a few miscellaneous improvements:
Which issue(s) this PR fixes:
(REQUIRED)
Closes #3250
Testing
Manually poking around