Skip to content

Filter Pop Up in Admin View#225

Open
rayyanmridha wants to merge 3 commits intomainfrom
rm-207-filter-pop-up
Open

Filter Pop Up in Admin View#225
rayyanmridha wants to merge 3 commits intomainfrom
rm-207-filter-pop-up

Conversation

@rayyanmridha
Copy link
Copy Markdown
Collaborator

@rayyanmridha rayyanmridha commented Apr 1, 2026

ℹ️ Issue

Closes #207

📝 Description

Used Claude Code to help debug and help with the final implementation

  1. Refactored the Status Pill to use a key value pair of variants (Record) making it easier and uniform with any usage of the component itself. A developer should have a much easier time calling upon a specific type of pill.
  2. Added the Filter pop up component
  3. Added the Filter pop up to the admin page
  4. Fixed up the padding of the buttons around the application table

✔️ Verification

bhchp-filter-pop-up.mov

🏕️ (Optional) Future Work / Notes

The Discipline Admin section of the filter pop up needs to be populated.

- refactor variants for status pills
- add filter pop up component
- fix application table scrolling in admin view
@rayyanmridha rayyanmridha changed the title Rm 207 filter pop up Filter Pop Up in Admin View Apr 1, 2026
@SamNie2027
Copy link
Copy Markdown
Collaborator

Also perhaps in a UX light it might make more sense to make that bottom scrollbar always visible in this view so that it gets strongly communicated to users that that component is scrollable.

Copy link
Copy Markdown
Collaborator

@SamNie2027 SamNie2027 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do really like the status pill consolidations

@SamNie2027
Copy link
Copy Markdown
Collaborator

Can you change the options for experiencetype to the actual backend enum? Same with rechecking other enum values for accurate options. And just redeclare those enums in the frontend with the same values and names as backend

@rayyanmridha
Copy link
Copy Markdown
Collaborator Author

rayyanmridha commented Apr 4, 2026

Also perhaps in a UX light it might make more sense to make that bottom scrollbar always visible in this view so that it gets strongly communicated to users that that component is scrollable.
@SamNie2027

Modern browsers tend to auto hide the scroll bar and even when trying to override it in CSS, the changes don't seem to work. This might also be in part due to a macOS feature but it might have to stay the way it is.

@rayyanmridha rayyanmridha requested a review from SamNie2027 April 4, 2026 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Admin Dashboard: Create filtering pop-up front-end

2 participants