> 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 TreeView.
> 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 changes

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/2095/files
  - new: https://git.openjdk.org/jfx/pull/2095/files/2efe738a..50d20691

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

  Stats: 32 lines in 1 file changed: 25 ins; 6 del; 1 mod
  Patch: https://git.openjdk.org/jfx/pull/2095.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/2095/head:pull/2095

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

Reply via email to