[Approved] RFR: 8233040: ComboBoxPopupControl: remove eventFilter for F4
Ajit Ghaisas
aghaisas at openjdk.org
Tue Nov 5 11:13:01 UTC 2019
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
Approved by aghaisas (Reviewer).
PR: https://git.openjdk.java.net/jfx/pull/28
More information about the openjfx-dev
mailing list