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

Align AI chat toggle keybinding with VS Code on macOS #14850

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

Conversation

cdamus
Copy link
Contributor

@cdamus cdamus commented Feb 5, 2025

What it does

On macOS, assign Control+Command+I keybinding and on other platforms Control+Alternate+I to align with VS Code on Mac and not clash with Chrome devtools on all platforms.

Fixes #14814

How to test

  1. On macOS,
    1. Hit ++I repeatedly and see the AI Chat view toggle on and off.
    2. Hit ++I and see the Chrome devtools appear.
  2. On Linux and Windows,
    1. Hit Ctrl+Alt+I repeatedly and see the AI Chat view toggle on and off.
    2. Hit whatever is the platform's keyboard shortcut for Chrome devtools and see them appear.

Review the keybinding editor. On macOS, you should see something like this (never mind that Theia shows the modifiers in the wrong order):

CleanShot 2025-02-05 at 13 36 42

Follow-ups

None.

Breaking changes

None.

Attribution

None.

Review checklist

Reminder for reviewers

On macOS, assign Control+Command+I keybinding and on other platforms
Control+Alternate+I to align with VS Code on Mac and not clash with
Chrome devtools on all platforms.

Fixes eclipse-theia#14814

Signed-off-by: Christian W. Damus <[email protected]>
@JonasHelming JonasHelming self-requested a review February 5, 2025 18:56
Copy link
Contributor

@JonasHelming JonasHelming left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM (on Linux)

@kittaakos Could you confirm this solves your issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Needs merge
Development

Successfully merging this pull request may close these issues.

Align AI chat window keybinding with VS Code defaults (on macOS)
2 participants