Skip to content

fix: handle meta keys in select key listener#2698

Merged
noahwaldner merged 2 commits intomainfrom
fix/minor-component-fixes
Apr 27, 2026
Merged

fix: handle meta keys in select key listener#2698
noahwaldner merged 2 commits intomainfrom
fix/minor-component-fixes

Conversation

@noahwaldner
Copy link
Copy Markdown
Contributor

  • Ignore key commands on Select when a meta key is held down
  • Use correct cursor for combobox element

@noahwaldner noahwaldner requested a review from a team as a code owner April 27, 2026 10:46
@noahwaldner noahwaldner requested a review from SamuelAlev April 27, 2026 10:46
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 27, 2026

🦋 Changeset detected

Latest commit: 21171fc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@frontify/fondue-components Patch
@frontify/fondue Patch
@frontify/fondue-rte Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 27, 2026

Deploy Preview for fondue-components ready!

Name Link
🔨 Latest commit 21171fc
🔍 Latest deploy log https://app.netlify.com/projects/fondue-components/deploys/69ef3eae5349f20008c0b0ed
😎 Deploy Preview https://deploy-preview-2698.components.fondue-components.frontify.com
📱 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.

// eslint-disable-next-line react-hooks/refs
{...getInputProps({
ref: inputCallbackRef,
onKeyDown: (event: React.KeyboardEvent) => {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
onKeyDown: (event: React.KeyboardEvent) => {
onKeyDown: (event: KeyboardEvent) => {

same for the others + "import from react" up

Copy link
Copy Markdown
Member

@SamuelAlev SamuelAlev left a comment

Choose a reason for hiding this comment

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

Small comment left to fix before merging, but to not slow you down, already approving.

@noahwaldner noahwaldner merged commit 8fd789c into main Apr 27, 2026
9 checks passed
@noahwaldner noahwaldner deleted the fix/minor-component-fixes branch April 27, 2026 13:30
@github-actions
Copy link
Copy Markdown
Contributor

Lead time: 2 hours, 49 minutes, 42 seconds (2.83 total hours) from first commit to close.
Review time: 2 hours, 43 minutes, 31 seconds (2.73 total hours) from ready for review to close.

  • First commit: 27.4.2026, 12:40:25.
  • Ready for review: 27.4.2026, 12:46:36.
  • Closed: 27.4.2026, 15:30:07.

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