Skip to content

Add Gender filter and active filter indicators to Horses grid#32

Merged
ovation22 merged 1 commit intomainfrom
feature/gender-filter-and-indicators
Jan 30, 2026
Merged

Add Gender filter and active filter indicators to Horses grid#32
ovation22 merged 1 commit intomainfrom
feature/gender-filter-and-indicators

Conversation

@ovation22
Copy link
Copy Markdown
Owner

Summary

  • Added Gender filter dropdown (All/Male/Female) to the Gender column options menu
  • Implemented active filters summary bar displaying all applied filters as removable badges
  • Each filter badge includes an individual clear button for quick removal
  • Added "Clear All" button to reset all filters at once
  • Gender filter integrates with existing AND/OR filter operator logic

Technical Details

  • Gender filter uses IsMale field with FilterOperator.Eq for boolean comparison
  • Filter state managed via bool? genderFilter for tri-state (null/true/false)
  • Active filters bar only displays when filters are active (HasActiveFilters())
  • Maintains existing debounce behavior for text-based filters
  • Column headers remain clickable for sorting and filter menu access

Test Plan

  • Verify Gender dropdown filters horses correctly (Male/Female/All)
  • Confirm active filters bar displays when any filter is applied
  • Test individual filter clear buttons
  • Test "Clear All" button removes all filters
  • Verify column headers remain clickable for sorting
  • Confirm filter integration with AND/OR operator logic

- Added Gender filter dropdown (All/Male/Female) to Gender column
- Implemented active filters summary bar showing all applied filters
- Each active filter displays as a badge with individual clear buttons
- Added "Clear All" button to remove all filters at once
- Gender filter uses IsMale field with Equals operator
- Maintains compatibility with existing AND/OR filter logic
@github-actions
Copy link
Copy Markdown

Test Results

723 tests  ±0   723 ✅ ±0   6s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit d7298fe. ± Comparison against base commit 0efae32.

@ovation22 ovation22 merged commit c1ac668 into main Jan 30, 2026
2 checks passed
@ovation22 ovation22 deleted the feature/gender-filter-and-indicators branch January 30, 2026 14:46
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.

1 participant