Skip to content

feat: add ESC key navigation support in settings menus#25

Merged
bvdr merged 1 commit intomasterfrom
feature/esc-navigation-24
Oct 3, 2025
Merged

feat: add ESC key navigation support in settings menus#25
bvdr merged 1 commit intomasterfrom
feature/esc-navigation-24

Conversation

@bvdr
Copy link
Owner

@bvdr bvdr commented Oct 3, 2025

Summary

Implements intuitive ESC key navigation throughout the settings menu system for improved user experience.

Changes

  • ✅ ESC in top-level settings menu exits settings
  • ✅ ESC in nested menus returns to parent menu
  • ✅ Settings menu loops after completing actions
  • ✅ Add explicit "← Exit Settings" and "← Back to Settings" options
  • ✅ Modified updateCredentials() to return navigation signals

Menu Navigation Flow

  1. Main Settings Menu → ESC exits / "← Exit Settings" option
  2. Credential Update Menu → ESC returns to settings / "← Back to Settings" option
  3. Provider Selection → ESC returns to settings
  4. Model Selection → ESC returns to settings
  5. After any action → Automatically returns to settings menu

Technical Implementation

  • Wrapped showSettings() in while(true) loop for continuous navigation
  • All isCancel() checks now use continue to return to parent menu
  • Added return value system ('back', 'continue') for submenu navigation
  • Global preferences object updates in real-time

Files Modified

  • index.js - Settings menu system with ESC navigation (+141 lines, -112 lines refactored)

Testing

  • ESC exits top-level settings menu
  • ESC returns from credential update submenu
  • ESC returns from provider/model selection
  • Explicit back/exit options work
  • Settings persist across menu loops

Fixes #24

Implement intuitive ESC key navigation throughout settings:
- ESC in top-level settings menu exits settings
- ESC in nested menus returns to parent menu
- Settings menu now loops after completing actions
- Add explicit exit/back options for better UX
- Modified updateCredentials() to return navigation signals

Improves user experience with standard navigation patterns.

Fixes #24
@bvdr bvdr merged commit 21e782c into master Oct 3, 2025
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.

Add ESC key navigation support in settings menus

1 participant