Skip to content

Cmd+Enter on a single inbox report opens Create PR dialog#1892

Draft
posthog[bot] wants to merge 2 commits intomainfrom
posthog-code/inbox-cmd-enter-create-pr
Draft

Cmd+Enter on a single inbox report opens Create PR dialog#1892
posthog[bot] wants to merge 2 commits intomainfrom
posthog-code/inbox-cmd-enter-create-pr

Conversation

@posthog
Copy link
Copy Markdown
Contributor

@posthog posthog Bot commented Apr 27, 2026

Summary

  • Adds a window-level keydown handler in ReportDetailPane that fires the same handleOpenCloudConfirm callback as the inline "Create PR" button.
  • Only active while a single report is selected (the pane is only mounted in that case) and the report is eligible (canCreateImplementationPr).
  • Skips when focus is in an input/textarea/select/contenteditable, or when a Radix dialog/popper is already open — so cmd+enter inside the cloud confirm dialog still goes to its TextArea.

Test plan

  • Select a single ready/pending-input report in the inbox; press cmd/ctrl+Enter → cloud confirm dialog opens.
  • Multi-select two reports; cmd+Enter does nothing (MultiSelectStack is shown, ReportDetailPane is unmounted).
  • Focus the report-list search input and press cmd+Enter → no dialog.
  • With the cloud confirm dialog open, cmd+Enter inside the prompt textarea does not re-open the dialog.

Created with PostHog Code

PostHog Code and others added 2 commits April 27, 2026 09:01
Mirrors the existing Create PR run-action button so the keyboard
shortcut and the click target stay in sync.

Generated-By: PostHog Code
Task-Id: 35931fe5-864a-4502-89a5-b0a98c8e7f21
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