> 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:

  Fix TreeTableView initial focus

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/2104/files
  - new: https://git.openjdk.org/jfx/pull/2104/files/f02ebd4d..6586a9ca

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

  Stats: 12 lines in 1 file changed: 7 ins; 2 del; 3 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