Skip to content

Fix Escape key mapping and add tests in parse-hotkey.ts#7928

Merged
rtivital merged 2 commits into
mantinedev:masterfrom
Han5991:feature/fix-hot-key-esc
Jun 16, 2025
Merged

Fix Escape key mapping and add tests in parse-hotkey.ts#7928
rtivital merged 2 commits into
mantinedev:masterfrom
Han5991:feature/fix-hot-key-esc

Conversation

@Han5991

@Han5991 Han5991 commented Jun 3, 2025

Copy link
Copy Markdown
Contributor

Description

fixed: #7923

This pull request addresses the following updates in parse-hotkey.ts:

  • Updated the Escape key mappings:
    • Changed 'Escape' and 'Esc' mappings to 'escape' for consistency.
    • Added 'esc' key to keyNameMap for backward compatibility.
  • Added comprehensive unit tests for Escape key handling to ensure correct functionality.

Han5991 added 2 commits June 3, 2025 09:40
…d add tests

- Change 'Escape' and 'Esc' mapping from 'esc' to 'escape' for consistency
- Add 'esc' key to keyNameMap for backward compatibility
- Add comprehensive tests for Escape key handling
@rtivital
rtivital merged commit d44e808 into mantinedev:master Jun 16, 2025
@rtivital

Copy link
Copy Markdown
Member

Thanks!

rtivital pushed a commit that referenced this pull request Jun 17, 2025
…tly in some browsers (#7928)

* [@matine/hooks] parse-hotkey.ts escapeHotkey fix add test code

* [@mantine/hooks] fix: update Escape key mapping in parse-hotkey.ts and add tests

- Change 'Escape' and 'Esc' mapping from 'esc' to 'escape' for consistency
- Add 'esc' key to keyNameMap for backward compatibility
- Add comprehensive tests for Escape key handling
@Han5991
Han5991 deleted the feature/fix-hot-key-esc branch June 22, 2025 10:00
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.

use-hotkeys with escape does not fire

2 participants