-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Search] Rework filter area #90
Conversation
2595ec8
to
7177e06
Compare
35b78fb
to
c4b10d8
Compare
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.
Thanks for the big work @cmoinier! Looks really nice!
I just had some issue using the range dropdown. Does this work on your side?
...p/search/search-filters/mel-datahub-dropdown-range/mel-datahub-dropdown-range.component.html
Show resolved
Hide resolved
...app/search/search-filters/mel-datahub-dropdown-range/mel-datahub-dropdown-range.component.ts
Outdated
Show resolved
Hide resolved
apps/datahub/src/app/search/search-filters/search-filters.component.html
Show resolved
Hide resolved
27f820b
to
5d7ea7e
Compare
68c6645
to
56c2536
Compare
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.
Thanks, works really well now! Feel free to merge once the CI is green.
Description
This PR reworks the filter area on the search page.
It now displays only 3 filters in "normal" mode, and all the filters, including two new ones (quality score and territories) on the "expanded" version. It also adds a reset button.
Since one of the new filters (quality score) is a range, a new dropdown selector component for number ranges is introduced.
Screenshots