-
Notifications
You must be signed in to change notification settings - Fork 348
More filter libraries options #866
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
Conversation
const dictionary = JSON.parse(source) as LibraryDictionaryModel; | ||
const allOptions = Object.keys(Object.values(dictionary)[0].libs[0].support); | ||
const indexAlgorithmStart = allOptions.findIndex( | ||
(option) => option == "hs256" |
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.
(option) => option == "hs256" | |
(option) => option === "hs256" |
Hello Javi. Looks great! just one thing. The "All" programming language filter doesn't work as expected. It doesn't display any libraries. Maybe we're considering the "all" query parameter as a programming language? We should add a check for that. |
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 good! Thank you Javi. One future improvement could be splitting the filters into different dropdowns. Let me know what you think.
Sounds a good improvement for the future, I'll create a ticket to address it and discuss with the UI team 👍 |
The library filter dropdown now has grouped categories for:
- Programming Language

- Algorithm

- Support
