Skip to content

Add UNICODE_TAP_DELAY to allow a different tap delay for Unicode - #26361

Draft
NapOli1084 wants to merge 1 commit into
qmk:developfrom
NapOli1084:unicode-tap-delay
Draft

Add UNICODE_TAP_DELAY to allow a different tap delay for Unicode#26361
NapOli1084 wants to merge 1 commit into
qmk:developfrom
NapOli1084:unicode-tap-delay

Conversation

@NapOli1084

Copy link
Copy Markdown
Contributor

Description

Adds a new UNICODE_TAP_DELAY define that allows configuring a different tap delay for Unicode.
It defaults to TAP_CODE_DELAY, which means the default behavior is unchanged.
Users may define it in their config.h to configure a different delay.

Note: I haven't updated the doc and tests yet, I'll do so if we agree on the solution.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@github-actions github-actions Bot added the core label Jul 24, 2026
Comment thread quantum/unicode/unicode.c
// correctly in the shifted case.
if (unicode_config.input_mode == UNICODE_MODE_LINUX && unicode_saved_led_state.caps_lock) {
tap_code(KC_CAPS_LOCK);
tap_code(KC_CAPS_LOCK); // Note: uses TAP_HOLD_CAPS_DELAY

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I kept KC_CAPS_LOCK taps as-is to avoid changing the current behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant