Skip to content

fix: settings shortcuts#199

Merged
Dimillian merged 2 commits intoDimillian:mainfrom
tobihagemann:fix-settings-shortcuts
Jan 22, 2026
Merged

fix: settings shortcuts#199
Dimillian merged 2 commits intoDimillian:mainfrom
tobihagemann:fix-settings-shortcuts

Conversation

@tobihagemann
Copy link
Copy Markdown
Contributor

Summary

  • add Cmd+, accelerator for Settings menu item
  • close Settings on Cmd+W and Escape
  • cover shortcut closes with SettingsView tests

Testing

  • npm run test

@Dimillian Dimillian merged commit 52e068f into Dimillian:main Jan 22, 2026
5 checks passed
@tobihagemann tobihagemann deleted the fix-settings-shortcuts branch January 22, 2026 16:46
gersmann pushed a commit to gersmann/codex-monitor-web that referenced this pull request Mar 10, 2026
Co-authored-by: Thomas Ricouard <ricouard77@gmail.com>
zip700 pushed a commit to zip700/CodexMonitor that referenced this pull request Apr 3, 2026
Co-authored-by: Thomas Ricouard <ricouard77@gmail.com>
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