Поведение диалога Assign изменилось за ночь: теперь требуется использование мыши

zogstrip · 10.Декабрь.2025 15:20:40

Modals are really hard to get right

Hopefully that fixes it and doesn’t introduce any regressions

github.com/discourse/discourse

FIX: Allow Enter key to work inside select-kit within modals

main ← fix-pressing-enter-within-select-kit-within-a-modal-inception-style

opened 03:19PM - 10 Dec 25 UTC

ZogStriP

+58 -2

When selecting an item in a select-kit dropdown (e.g., the assignee chooser in t…he assign modal), pressing Enter would close the modal without completing the selection. This was caused by commit 1d2ae5159b which added `{ capture: true }` to the modal's keydown listener to prevent keypresses from leaking outside modals. However, this caused the modal to intercept Enter before the select-kit could process it, triggering the primary button click prematurely. Add a check for `.select-kit` elements in `shouldTriggerClickOnEnter()` so the modal defers to select-kit's own Enter key handling. Ref - meta/t/390910

Тема		Ответов	Просм.
Key combo to submit assignment dialog, please Feature assign	16	886	12.07.2025
Can no longer submit from assignment comment :cry: Bug assign	2	235	25.04.2024
Focus in 'Edit Assignment' dialog UX assign	4	433	31.03.2026
Application-level keyboard shortcuts should be disabled when a modal is open Bug keyboard-shortcuts	2	772	18.01.2018
Assignment dialog never gives me the chance to leave a note Feature assign	12	611	10.06.2022

Поведение диалога Assign изменилось за ночь: теперь требуется использование мыши

Связанные темы