> This PR alters the way ComboBox and Spinner deliver KeyEvents to their > TextField editors. When a ComboBox or Spinner is the focus owner it is the > target of all key events. Currently the skin installs a filter to catch key > events and re-fire most of them at the TextEdit. The skin copies the event, > fires the copy at the TextField, and then consumes the original event. This > confuses the system menu bar logic on macOS; only the original event can > trigger a menu item and that event is always being consumed. > > In this PR only the original key event makes its way up and down the event > dispatch chain. To drive the TextField the skin delivers the event copy > directly to the TextField's event dispatcher and only consumes the original > event if the TextField consumes the copy. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai).
Martin Fox has updated the pull request incrementally with one additional commit since the last revision: Clean up based on review feedback ------------- Changes: - all: https://git.openjdk.org/jfx/pull/2166/files - new: https://git.openjdk.org/jfx/pull/2166/files/75af2a5a..2cf952e3 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=2166&range=01 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2166&range=00-01 Stats: 4 lines in 3 files changed: 1 ins; 0 del; 3 mod Patch: https://git.openjdk.org/jfx/pull/2166.diff Fetch: git fetch https://git.openjdk.org/jfx.git pull/2166/head:pull/2166 PR: https://git.openjdk.org/jfx/pull/2166
