Assign dialog behavior changed overnight to require mousing

zogstrip · 10 december 2025 om 15:20

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

Topic		Antwoorden	Weergaven
Key combo to submit assignment dialog, please Feature assign	16	884	12 juli 2025
Can no longer submit from assignment comment :cry: Bug assign	2	234	25 april 2024
Focus in 'Edit Assignment' dialog UX assign	4	430	31 maart 2026
Application-level keyboard shortcuts should be disabled when a modal is open Bug keyboard-shortcuts	2	772	18 januari 2018
Assignment dialog never gives me the chance to leave a note Feature assign	12	606	10 juni 2022

Assign dialog behavior changed overnight to require mousing

Gerelateerde topics