On Wed, 16 Oct 2019 13:09:35 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
> On Wed, 16 Oct 2019 13:07:45 GMT, Jeanette Winzenburg <faste...@openjdk.org> > wrote: > >> This is a fix for https://bugs.openjdk.java.net/browse/JDK-8207759 >> >> The issue is that default/cancel button on a scene are triggered even though >> a onKeyPressed handler for ENTER/CANCEL consumes the keyEvent. See the bug >> for details on both cause and fix. >> >> There are additional/changed tests to verify the fix. All old tests are >> passing. >> >> ---------------- >> >> Commits: >> - 93556393: Fix: VK_ENTER not consumed by TextField when default Button >> exists >> >> Changes: https://git.openjdk.java.net/jfx/pull/15/files >> Webrev: https://webrevs.openjdk.java.net/jfx/15/webrev.00 >> Issue: https://bugs.openjdk.java.net/browse/JDK-8207759 >> Stats: 553 lines in 8 files changed: 528 ins; 20 del; 5 mod >> Patch: https://git.openjdk.java.net/jfx/pull/15.diff >> Fetch: git fetch https://git.openjdk.java.net/jfx pull/15/head:pull/15 > > @aghaisas can you review this as well? This will need two reviewers. PR: https://git.openjdk.java.net/jfx/pull/15