> 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 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 four additional commits since the last revision: - Merge branch 'openjdk:master' into 8379662.TreeTableViewFocus - Fix TreeTableView initial focus - Minor changes - Fix TreeTableView focus and added test ------------- Changes: - all: https://git.openjdk.org/jfx/pull/2104/files - new: https://git.openjdk.org/jfx/pull/2104/files/6586a9ca..c2f19ac5 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=2104&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2104&range=01-02 Stats: 3677 lines in 41 files changed: 1816 ins; 641 del; 1220 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
