RFR: 8313138: Horizontal Scrollbar Keyboard enhancement [v6]

Andy Goryachev angorya at openjdk.org
Mon May 6 22:00:27 UTC 2024


> Adding alt-ctrl-LEFT/RIGHT (option-command-LEFT/RIGHT) key bindings to
> 
> - ListView
> - TreeView
> - TableView
> - TreeTableView
> 
> to support keyboard-only horizontal scrolling.  The main reason for the change is to improve accessibility.
> 
> **NOTE: For controls in right-to-left orientation, the direction is reversed.**
> 
> As far as I can tell, these key combinations do not interfere with editing.
> 
> The proposed solution can be further optimized by adding a public method to the VirtualFlow class, something like
> 
> 
> public void horizontalUnitScroll(boolean right);
> 
> 
> Q: Does this change require a CSR to explain the change in the controls' behavior?  We don't yet have the key bindings documented in /doc-files/behavior
> 
> Note:
> Jenkins headful test passed on all mac configurations, failed on all linux configurations (master branch failed also, so it is test issue), while windows configuration is not yet available.

Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 16 additional commits since the last revision:

 - added vertical scrolling
 - Merge branch 'master' into 8313138.horizontal
 - tests
 - cleanup
 - node orientation
 - Merge remote-tracking branch 'origin/master' into 8313138.horizontal
 - table view behavior
 - tree view behavior
 - list view behavior
 - orientation
 - ... and 6 more: https://git.openjdk.org/jfx/compare/969d0b8e...859f7bbd

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/1393/files
  - new: https://git.openjdk.org/jfx/pull/1393/files/5bae5e7a..859f7bbd

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=1393&range=05
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1393&range=04-05

  Stats: 5171 lines in 211 files changed: 4184 ins; 503 del; 484 mod
  Patch: https://git.openjdk.org/jfx/pull/1393.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1393/head:pull/1393

PR: https://git.openjdk.org/jfx/pull/1393


More information about the openjfx-dev mailing list