如何让JavaFX TreeView在节点扩展时表现一致？

Question

我有一个TreeView带有隐形根的JavaFX 和一些"文件夹" TreeItems,它们有许多"文件" TreeItems作为子项.'文件夹' TreeItems通常适合TreeView没有任何滚动条.

invisible-root/
    folder/
    folder/
    folder/
        file
        file
        file
        ...
        file

有时,当我展开"文件夹"时TreeItem,会出现滚动条但滚动位置保持不变. (这就是我想要的!)但是,有时,扩展a TreeItem导致滚动条出现并且TableView 滚动到扩展的最后一个子节点TreeItem!

这是非常意外和令人惊讶的,特别是因为我很难预测我将看到的两种行为中的哪一种:(1)保持放置,或(2)滚动到最后一项.就个人而言,我认为行为(1)不那么令人惊讶和可取.

有关如何处理这个的任何想法？

我在Java8u31上看到了这种行为.

Answer 1

问题出在 VirtualFlow 中。在layoutChildren()中有这个部分：

 if (lastCellCount != cellCount) {
            // The cell count has changed. We want to keep the viewport
            // stable if possible. If position was 0 or 1, we want to keep
            // the position in the same place. If the new cell count is >=
            // the currentIndex, then we will adjust the position to be 1.
            // Otherwise, our goal is to leave the index of the cell at the
            // top consistent, with the same translation etc.
            if (position == 0 || position == 1) {
                // Update the item count
//                setItemCount(cellCount);
            } else if (currentIndex >= cellCount) {
                setPosition(1.0f);
//                setItemCount(cellCount);
            } else if (firstCell != null) {
                double firstCellOffset = getCellPosition(firstCell);
                int firstCellIndex = getCellIndex(firstCell);
//                setItemCount(cellCount);
                adjustPositionToIndex(firstCellIndex);
                double viewportTopToCellTop = -computeOffsetForCell(firstCellIndex);
                adjustByPixelAmount(viewportTopToCellTop - firstCellOffset);
            }

如果位置为 1.0（== 滚动到底部），就会出现问题，因为在这种情况下不会重新计算。解决方法是覆盖 TreeViewSkin 以提供您自己的 VirtualFlow 并修复那里的行为。

下面的代码旨在说明问题，它不是真正的解决方案，如果您确实想修复它，它只是一个起点：

import com.sun.javafx.scene.control.skin.TreeViewSkin;
import com.sun.javafx.scene.control.skin.VirtualFlow;
import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.control.IndexedCell;
import javafx.scene.control.Skin;
import javafx.scene.control.TreeItem;
import javafx.scene.control.TreeView;
import javafx.scene.layout.StackPane;
import javafx.stage.Stage;

public class TreeViewScrollBehaviour extends Application {

    @Override
    public void start(Stage primaryStage) {

        TreeView treeView = new TreeView() {
            @Override
            protected Skin createDefaultSkin() {
                return new TTreeViewSkin(this); //To change body of generated methods, choose Tools | Templates.
            }

        };
        TreeItem<String> treeItem = new TreeItem<String>("Root");
        for (int i = 0; i < 20; i++) {
            TreeItem<String> treeItem1 = new TreeItem<>("second layer " + i);
            treeItem.getChildren().add(treeItem1);
            for (int j = 0; j < 20; j++) {
                treeItem1.getChildren().add(new TreeItem<>("Third Layer " + j));

            }

        }

        treeView.setRoot(treeItem);
        StackPane root = new StackPane();
        root.getChildren().addAll(treeView);
        Scene scene = new Scene(root, 300, 250);
        primaryStage.setTitle("Hello World!");
        primaryStage.setScene(scene);
        primaryStage.show();
    }

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        launch(args);
    }

    class TTreeViewSkin<T extends IndexedCell> extends TreeViewSkin<T> {

        public TTreeViewSkin(TreeView treeView) {
            super(treeView);
        }

        @Override
        protected VirtualFlow createVirtualFlow() {
            return new TVirtualFlow<T>(); //To change body of generated methods, choose Tools | Templates.
        }

    }

    class TVirtualFlow<T extends IndexedCell> extends VirtualFlow<T> {

        @Override
        public double getPosition() {
            double position = super.getPosition();
            if (position == 1.0d) {
                return 0.99999999999;
            }
            return super.getPosition(); //To change body of generated methods, choose Tools | Templates.
        }

        @Override
        public void setPosition(double newPosition) {
            if (newPosition == 1.0d) {
                newPosition = 0.99999999999;
            }
            super.setPosition(newPosition); //To change body of generated methods, choose Tools | Templates.
        }

    }
}