Skip to content

fix(seer): use standard hotkey for minimize behavior, remove tab hijacking#112993

Merged
natemoo-re merged 4 commits intomasterfrom
fix/seer-hotkeys
Apr 15, 2026
Merged

fix(seer): use standard hotkey for minimize behavior, remove tab hijacking#112993
natemoo-re merged 4 commits intomasterfrom
fix/seer-hotkeys

Conversation

@natemoo-re
Copy link
Copy Markdown
Member

Seer panel currently hijacks the Tab key to act as a hotkey when the panel is minimized, breaking keyboard-based navigation. Since Tab is the native "move focus" keybinding, keyboard users are effectively trapped in the Seer panel with no way to navigate to other focusable elements on the page.

This PR updates the implementation to use the normal Cmd+/ keybinding to open the panel when it's minimized. It also removes the stateful placeholder since typing any message will refocus the textarea.

@natemoo-re natemoo-re requested a review from a team April 14, 2026 22:31
@natemoo-re natemoo-re requested a review from a team as a code owner April 14, 2026 22:31
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 14, 2026
@sentry
Copy link
Copy Markdown
Contributor

sentry bot commented Apr 14, 2026

Sentry Snapshot Testing

Name Added Removed Modified Renamed Unchanged Status
sentry-frontend
sentry-frontend
0 0 0 0 204 ✅ Unchanged

Comment thread static/app/views/seerExplorer/panelContainers.tsx Outdated
Copy link
Copy Markdown
Member

@JonasBa JonasBa left a comment

Choose a reason for hiding this comment

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

Thank you 🙏🏼

@natemoo-re natemoo-re merged commit 5910df7 into master Apr 15, 2026
64 checks passed
@natemoo-re natemoo-re deleted the fix/seer-hotkeys branch April 15, 2026 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants