Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(desktop-ui): Command/ctrl + comma goes to Settings view #1439

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

matthewdiamant
Copy link
Contributor

@matthewdiamant matthewdiamant commented Feb 28, 2025

A very standard shortcut is Command + comma (or Ctrl + comma) to go to an App's settings or preferences (try it out on Slack, Notion, Discord, or your favorite app).

Also adding Settings to the app menu, after the About item, which is also where Chrome/Notion/Slack has it.
image

Also, adding the general ability to change views with the 'set-view' message.

@matthewdiamant matthewdiamant force-pushed the mdiamant/command-comma-settings branch from 6ee3eb5 to 42fa72d Compare February 28, 2025 04:58
Copy link
Collaborator

@baxen baxen left a comment

Choose a reason for hiding this comment

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

Looks good! just one small comment on the menu label

@@ -228,7 +228,7 @@ export default function MoreMenu({ setView }: { setView: (view: View) => void })
}}
className="w-full text-left p-2 text-sm hover:bg-bgSubtle transition-colors"
>
Settings
Settings (cmd+,)
Copy link
Collaborator

Choose a reason for hiding this comment

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

tiny nit: would show this as ⌘,

(or i think electron has a helper to do ⌘/^ depending if on windows? but i can't seem to find it and this is something we can review later when we have more official windows support)

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