Skip to content

Desktop: Handle keyboard shortcuts that are skipped by CEF on Mac#4322

Open
timon-schelling wants to merge 2 commits into
desktop-ui-cratefrom
desktop-mac-fix-text-shortcuts
Open

Desktop: Handle keyboard shortcuts that are skipped by CEF on Mac#4322
timon-schelling wants to merge 2 commits into
desktop-ui-cratefrom
desktop-mac-fix-text-shortcuts

Conversation

@timon-schelling

Copy link
Copy Markdown
Member

fixes #3516
depends on #4321

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a macOS-specific keyboard handler to support standard editing shortcuts (such as Cmd+A, Cmd+C, Cmd+V, etc.) and document navigation. Feedback on the implementation highlights an eager return bug in the command shortcut block that prevents other command shortcuts from being handled, as well as a non-standard shortcut mapping for document-level selection (using Option+Shift instead of Cmd+Shift). A refactoring suggestion is provided to address both issues.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread desktop/ui/src/internal/keyboard_handler.rs Outdated
@Keavon Keavon changed the title Desktop: Handle keyboard shourtcuts that are skiped by CEF on Mac Desktop: Handle keyboard shortcuts that are skipped by CEF on Mac Jul 10, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 3 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@timon-schelling

timon-schelling commented Jul 10, 2026

Copy link
Copy Markdown
Member Author

!build desktop (Run ID 29123173272)

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown
📦 Linux Build Complete for 04074bc
Download binary
Download Flatpak

@github-actions

Copy link
Copy Markdown
📦 Mac Build Complete for 04074bc
Download binary

@github-actions

Copy link
Copy Markdown
📦 Windows Build Complete for 04074bc
Download binary

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