Skip to content

fix: hide tab shortcut hints when Shift is held (Cmd+Shift+5 screenshots)#4214

Merged
ComputelessComputer merged 1 commit intomainfrom
devin/1771947578-hide-tab-shortcuts-during-screenshot
Feb 24, 2026
Merged

fix: hide tab shortcut hints when Shift is held (Cmd+Shift+5 screenshots)#4214
ComputelessComputer merged 1 commit intomainfrom
devin/1771947578-hide-tab-shortcuts-during-screenshot

Conversation

@devin-ai-integration
Copy link
Contributor

Summary

When taking macOS screenshots with Cmd+Shift+5, the Cmd key triggers useCmdKeyPressed, causing ⌘ 1, ⌘ K, etc. shortcut badges to appear in tabs and search — and get captured in the screenshot.

This fix suppresses the shortcut hints whenever Shift is held alongside Cmd (setIsCmdPressed(!e.shiftKey)), and restores them if Shift is released while Cmd is still held.

Review & Testing Checklist for Human

  • Test on macOS: Press Cmd+Shift+5 and confirm shortcut badges do NOT appear in tabs/search
  • Test Cmd alone: Press and hold Cmd — shortcut badges should still appear as before
  • Test Cmd+Shift → release Shift: Hold Cmd+Shift (no badges), release Shift while keeping Cmd held — badges should reappear
  • Verify ⌘ K search hint: Confirm the search bar shortcut hint also respects this behavior

Notes

…ing Cmd+Shift+5 screenshots)

Co-Authored-By: John <john@hyprnote.com>
@netlify
Copy link

netlify bot commented Feb 24, 2026

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit 5792f46
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/699dc6957f913b00085c36ac
😎 Deploy Preview https://deploy-preview-4214--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@netlify
Copy link

netlify bot commented Feb 24, 2026

Deploy Preview for hyprnote-storybook ready!

Name Link
🔨 Latest commit 5792f46
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/699dc6950fd6870008e5d5ea
😎 Deploy Preview https://deploy-preview-4214--hyprnote-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ComputelessComputer ComputelessComputer merged commit 7fff09f into main Feb 24, 2026
11 checks passed
@ComputelessComputer ComputelessComputer deleted the devin/1771947578-hide-tab-shortcuts-during-screenshot branch February 24, 2026 15:48
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