> The bug occurs when showRoot is set to false on a root with no children, the 
> expanded item count drops is 0 causing `isFocused(0)` to return false even 
> though `focusedIndex` is still 0. When items are added afterwards the 
> `treeItemListener` sees `focusedIndex=0` and incorrectly shifts it to 1 
> placing the focus on second item of the `TreeTableView`.
> The fix replaces `isFocused(0)` with `getFocusedIndex() >= 0`, which reads 
> the raw stored index.

Ziad El Midaoui has updated the pull request incrementally with one additional 
commit since the last revision:

  Minor test changes

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/2104/files
  - new: https://git.openjdk.org/jfx/pull/2104/files/c2f19ac5..f32087ab

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=2104&range=03
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2104&range=02-03

  Stats: 12 lines in 2 files changed: 0 ins; 5 del; 7 mod
  Patch: https://git.openjdk.org/jfx/pull/2104.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/2104/head:pull/2104

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

Reply via email to