Skip to content

Conversation

Nerivec
Copy link
Owner

@Nerivec Nerivec commented Jul 25, 2025

  • all table search fields will be independently persisted to local storage
  • dashboard search field will be persisted to local storage
  • fix a few sort/filter issues
  • add clear buttons to table search fields

Fixes #124

@Nerivec Nerivec requested review from Koenkk and Copilot July 25, 2025 18:08
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements persistent search functionality across the application by storing table search fields and dashboard search in local storage, while also fixing several sort/filter issues and adding clear buttons to search fields.

  • Adds independent local storage persistence for all table search fields and dashboard search
  • Implements clear buttons for table search fields with proper UI styling
  • Fixes sorting functionality for availability states in the devices table

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/pages/TouchlinkPage.tsx Adds filterFn: "includesString" to table columns for proper search functionality
src/pages/OtaPage.tsx Adds filterFn: "includesString" to device and model columns
src/pages/GroupsPage.tsx Adds filterFn: "includesString" to ID and friendly name columns
src/pages/DevicesPage.tsx Adds column filters and implements custom sorting for availability states
src/pages/Dashboard.tsx Implements persistent search using local storage with useEffect hook
src/localStoreConsts.ts Defines new constants for table and dashboard filter storage keys
src/components/table/TextFilter.tsx Adds persistence, clear button functionality, and improved UI layout
src/components/table/Table.tsx Integrates storage key generation for individual column filters
package.json Version bump from 1.4.2 to 1.5.0

@Nerivec Nerivec merged commit f24c0d9 into main Jul 26, 2025
4 checks passed
@Nerivec Nerivec deleted the persist-searches branch July 26, 2025 15:17
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.

Table search not persisting navigation
2 participants