[Rev 01] RFR: 8235480: Regression: [RTL] Arrow keys navigation doesn't respect TableView orientation

Ajit Ghaisas aghaisas at openjdk.java.net
Mon Feb 24 17:07:48 UTC 2020


> Bug : https://bugs.openjdk.java.net/browse/JDK-8235480
> 
> Fix : Added the missed out RTL checks to the key mappings in TableViewBehaviorBase class.
> 
> Testing : Modified unit tests in TableViewKeyInputTest to take orientation as a parameter. The Left/Right key press tests have been modified to address LTR and RTL orientations.
> 
> Note : If this test modification is acceptable, I would like to address other similar tests separately. (I will create a test JBS issue and address later)

The pull request has been updated with 1 additional commit.

-------------

Added commits:
 - cbbb884f: Support dynamic Node orientation change

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/114/files
  - new: https://git.openjdk.java.net/jfx/pull/114/files/5f8a925b..cbbb884f

Webrevs:
 - full: https://webrevs.openjdk.java.net/jfx/114/webrev.01
 - incr: https://webrevs.openjdk.java.net/jfx/114/webrev.00-01

  Stats: 91 lines in 2 files changed: 70 ins; 3 del; 18 mod
  Patch: https://git.openjdk.java.net/jfx/pull/114.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/114/head:pull/114

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


More information about the openjfx-dev mailing list