RFR: 8233040: ComboBoxPopupControl: remove eventFilter for F4

Ajit Ghaisas aghaisas at openjdk.org
Mon Nov 4 12:46:37 UTC 2019

On Thu, 31 Oct 2019 21:20:55 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> On Thu, 31 Oct 2019 15:04:35 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:
>> https://bugs.openjdk.java.net/browse/JDK-8233040
>> The issue is that the eventFilter in CBPC uses F4 for toggling the popup _and_ consumes it. This prevents client code to register there own filter.
>> The fix was to remove the block entirely - F4 is already handled by a keyMapping in ComboBoxBehaviorBase. Added test that fails before the change and passes after, old tests are all passing.
>> ----------------
>> Commits:
>>  - 2f015406: Fix: ComboBoxPopupControl - remove eventFilter for F4
>> Changes: https://git.openjdk.java.net/jfx/pull/28/files
>>  Webrev: https://webrevs.openjdk.java.net/jfx/28/webrev.00
>>   Issue: https://bugs.openjdk.java.net/browse/JDK-8233040
>>   Stats: 159 lines in 2 files changed: 153 ins; 6 del; 0 mod
>>   Patch: https://git.openjdk.java.net/jfx/pull/28.diff
>>   Fetch: git fetch https://git.openjdk.java.net/jfx pull/28/head:pull/28
> The PR title should match the bug title, so:
> 8233040: ComboBoxPopupControl: remove eventFilter for F4
> @aghaisas can you review this, and also comment on whether you feel it needs a second reviewer (probably not unless you find something during the review that invites a more careful review).

I can confirm that key F4 is managed even after removing this code block.

PR: https://git.openjdk.java.net/jfx/pull/28

More information about the openjfx-dev mailing list