Adding filters to the application view#22
Conversation
There was a problem hiding this comment.
I pulled your branch locally and tested the status filter functionality.
The feature works, but I noticed a few behavior points that could be refined to better align with the intended filter behavior.
Observed points:
• When selecting the “closed” status filter and there are no closed applications, the applications list is not updated/reflected correctly.
• After selecting a status filter, typing in the search input causes the previously selected status filter to be ignored.
• The inverse also happens: when searching first and then applying a status filter, the search input is no longer considered.
In addition, I made some local UX/UI improvements, including refinements to the button styling, dropdown behavior, and animations, to make the interaction clearer and more fluid.
With these adjustments, the status filter and search work together consistently, and the overall user experience improves.
We can proceed in one of two ways — whichever you prefer:
- I can push my commits directly to your branch (if you’re okay with that), or
- I can explain the main changes so you can apply them to the PR.
Let me know what you prefer so we can move forward in the best way.
…e-filtering, memoize lists; improve filter button UX/animation (#23)
|
Thanks for the collaboration on this. The refinement points discussed were addressed, and the status filter and search behavior are now aligned as expected. Everything looks good - moving forward with the merge. |




Summary
Type of Change
Included Changes
Features
Testing
Checklist