Skip to content

feat(home): add keyboard-first tool search above the category pills - #102

Merged
McanKul merged 1 commit into
McanKul:developmentfrom
nonamexishere:feat/73-home-tool-search
Sep 10, 2026
Merged

McanKul merged 1 commit into
McanKul:developmentfrom
nonamexishere:feat/73-home-tool-search

Conversation

@nonamexishere

Copy link
Copy Markdown
Contributor

Summary

Adds a prominent tool search box on the desktop home screen, above the category pills. It filters the existing 23-tool registry (name, description, category, aliases) via the searchTools helper from #92. Ctrl/Cmd+K focuses the field on Home; Escape clears then blurs; a visible clear control clears and keeps focus. Non-empty queries hide the featured strip and show matches in the grid (including Merge/Compress/Reorder when they match). Focus uses the warm/orange accent in both themes.

Fixes #73.

Why

Home listed 23 tools with category pills only. .tool-search styles existed but nothing was wired. Searching tools (not documents or the filesystem) is the remaining half of #73 after #92 landed the matcher.

Validation

  • npm test (312 passed)
  • npm run typecheck
  • cargo test --manifest-path src-tauri/Cargo.toml --lib
  • Manual: type docx / merge, category ∩ query, no-results, ✕ keeps focus, Escape clears+blurs, ⌘K on Home only, orange focus in light and dark

Privacy Checklist

  • This keeps OffPDF usable offline.
  • This does not upload, log, or transmit user files.
  • New dependencies or bundled binaries have compatible licenses. (none added)

Wire the existing searchTools helper into a prominent home search box.
Ctrl/Cmd+K focuses it; Escape clears then blurs; a visible clear control
keeps focus. Non-empty queries hide the featured strip and filter the
full grid. Focus uses the warm accent in both themes.

@McanKul McanKul left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks — focused implementation, useful keyboard behavior, and solid coverage.

@McanKul
McanKul merged commit 015815e into McanKul:development Sep 10, 2026
2 checks passed
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.

2 participants