Skip to content

fix: command palette navigation and reduce window height#67

Merged
vuon9 merged 1 commit intomainfrom
fix/command-palette-navigation
Mar 29, 2026
Merged

fix: command palette navigation and reduce window height#67
vuon9 merged 1 commit intomainfrom
fix/command-palette-navigation

Conversation

@vuon9
Copy link
Copy Markdown
Owner

@vuon9 vuon9 commented Mar 29, 2026

Summary

  • Fix command palette navigation by moving listener from NavigationHandler component directly into App component
  • Use Events.On from @wailsio/runtime for proper event handling between Go backend and React frontend
  • Reduce command palette window height to 80% (480px → 384px) for better UX
  • Adjust CSS heights for results area (332px → 264px) and list (320px → 252px) to match

Test Plan

  • Open command palette with Cmd+Shift+Space
  • Select a tool and verify navigation works
  • Verify window height is reduced by ~20%
  • Run Go tests: go test ./...

@vuon9 vuon9 enabled auto-merge (squash) March 29, 2026 14:15
- Fix navigation listener by moving from NavigationHandler to App component
- Use Events.On from @wailsio/runtime for proper event handling
- Reduce command palette window height to 80% (480px -> 384px)
- Adjust CSS heights for results area and list to match new window size
@vuon9 vuon9 force-pushed the fix/command-palette-navigation branch from 3fa8837 to 1a614a2 Compare March 29, 2026 14:18
@github-actions
Copy link
Copy Markdown

Code Coverage

Package Line Rate Health
devtoolbox/internal/barcode 90%
devtoolbox/internal/codeformatter 75%
devtoolbox/internal/converter 60%
devtoolbox/internal/datagenerator 79%
devtoolbox/internal/datetimeconverter 74%
devtoolbox/internal/jwt 42%
devtoolbox/internal/numberconverter 0%
devtoolbox/internal/settings 0%
Summary 65% (3410 / 5275)

Minimum allowed line rate is 60%

@vuon9 vuon9 merged commit d78a5f8 into main Mar 29, 2026
2 checks passed
@vuon9 vuon9 deleted the fix/command-palette-navigation branch March 29, 2026 14:19
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