Skip to content

Prevent tag selection on Ctrl/Cmd+Enter to allow save action#22

Merged
pheuberger merged 2 commits into
mainfrom
claude/fix-bookmark-tag-save-isBqD
Feb 5, 2026
Merged

Prevent tag selection on Ctrl/Cmd+Enter to allow save action#22
pheuberger merged 2 commits into
mainfrom
claude/fix-bookmark-tag-save-isBqD

Conversation

@pheuberger
Copy link
Copy Markdown
Owner

Summary

Modified the TagInput component to prevent tag selection and creation when Ctrl/Cmd+Enter is pressed, allowing the keyboard shortcut to bubble up for save actions instead.

Changes

  • Updated Enter key handler to check for ctrlKey and metaKey modifiers before selecting a tag from the dropdown
  • Updated Enter key handler to check for ctrlKey and metaKey modifiers before creating a new tag from input
  • Added clarifying comments explaining the intent of these checks

Details

Previously, pressing Ctrl/Cmd+Enter in the TagInput would trigger tag selection/creation, preventing the common save shortcut from working at the application level. Now, when either modifier key is held, the Enter key event is allowed to propagate up the component tree, enabling parent components to handle the save action while still supporting regular Enter key functionality for tag operations.

https://claude.ai/code/session_01XANuHfRC6v45FxosHFmxYT

When pressing Ctrl+Enter (or Cmd+Enter on Mac) in the tag selection
dropdown, the save action should be triggered without selecting the
currently highlighted tag. This makes the behavior more intuitive -
Ctrl+Enter means "save as-is" regardless of what's highlighted.

https://claude.ai/code/session_01XANuHfRC6v45FxosHFmxYT
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 5, 2026

Deploy Preview for hypermarkk ready!

Name Link
🔨 Latest commit 9f537dc
🔍 Latest deploy log https://app.netlify.com/projects/hypermarkk/deploys/69843cb450b70600087b6055
😎 Deploy Preview https://deploy-preview-22--hypermarkk.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.

@pheuberger pheuberger merged commit c4b261f into main Feb 5, 2026
8 checks passed
@pheuberger pheuberger deleted the claude/fix-bookmark-tag-save-isBqD branch February 5, 2026 09:10
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.

2 participants