Skip to content

Conversation

michelroegl-brunner
Copy link
Member

Summary

This PR adds comprehensive sorting functionality to the installed scripts table in the InstalledScriptsTab component.

Changes Made

  • ✅ Added sortable columns for Script Name, Container ID, Server, Status, and Installation Date
  • ✅ Implemented clickable table headers with visual sort indicators (↑/↓)
  • ✅ Added ascending/descending toggle functionality
  • ✅ Maintained existing search and filter functionality
  • ✅ Set default sort by Script Name (ascending)
  • ✅ Added proper handling of null values in sorting logic

Features

  • Interactive Headers: Click any column header to sort by that column
  • Visual Feedback: Sort indicators show current sort field and direction
  • Smart Sorting: Case-insensitive text sorting, proper date sorting, null value handling
  • Seamless Integration: Works with existing search and filter functionality

Testing

  • All columns sort correctly in both ascending and descending order
  • Sorting works with filtered results
  • Null values are handled gracefully
  • UI provides clear visual feedback

Screenshots

The table headers now show sort indicators and are clickable for easy sorting by any column.

- Add sortable columns for Script Name, Container ID, Server, Status, and Installation Date
- Implement clickable table headers with visual sort indicators
- Add ascending/descending toggle functionality
- Maintain existing search and filter functionality
- Default sort by Script Name (ascending)
- Handle null values gracefully in sorting logic
@michelroegl-brunner michelroegl-brunner requested a review from a team as a code owner October 9, 2025 07:47
@michelroegl-brunner michelroegl-brunner self-assigned this Oct 9, 2025
- Fix TypeScript ESLint errors in InstalledScriptsTab.tsx
- Replace || with ?? for safer null/undefined handling
- Build now passes successfully
@michelroegl-brunner michelroegl-brunner merged commit 4ed3e42 into main Oct 9, 2025
4 checks passed
@michelroegl-brunner michelroegl-brunner deleted the feat/sorting_installedtab branch October 14, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant