On Sat, 15 Feb 2020 14:54:50 GMT, Kevin Rushforth <[email protected]> wrote:
>> thanks, Kevin - that's what I tried, but it refused to accept the second ..
>> so sticking with single comments until I feel like digging into this
>> weirdness.
>
> Regarding the fix, it seems that you need to eliminate the `rtl` variable you
> added earlier, and move the test into the listener, like this:
>
> new KeyMapping(LEFT, e -> isRtl() ? selectRightCell() : selectLeftCell()),
@kevinrushforth, this new KeyMapping suggestion does not compile.
We need something like -
`new KeyMapping(LEFT, e -> { if(isRTL()) selectRightCell(); else
selectLeftCell();}),`
-------------
PR: https://git.openjdk.java.net/jfx/pull/114