-
Notifications
You must be signed in to change notification settings - Fork 76
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
LF-4226: Clear filters on reset #3218
Conversation
Thanks @zagran for taking a look here! Reading the Jira ticket , I think what I was expecting was that when the user "clears all" filters that it returns to the default state of "all selected" instead of "none selected". Its an awkward state because we want them all to be selected but not show the individual pills unless interacted with. |
I'll fix it then. Thank you for watching Do I need to change something in the code? Since this is my first contribution, the code may not be perfect (or even not correct at all) - I don't want to add more mess or workarounds |
I think yes the code should be corrected. I can do the request changes button haha but most times I do not use it unless I know what to change. |
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.
"All" should be selected when cleared. But no pills should be present.
Returned this ticket to the Backlog. I don't understand the final requirements. |
Description
Clear filter options, previously default value was used.
Jira link: https://lite-farm.atlassian.net/browse/LF-4226
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: