fix(tree): Modify the `shouldVirtualizeFlatTree` function to support virtualization for non-empty trees.
This commit is contained in:
parent
ab1b4ae227
commit
8cf0f612d4
|
|
@ -27,5 +27,5 @@ export function flattenTree(nodes: TreeNode[]): FlatTreeNode[] {
|
|||
}
|
||||
|
||||
export function shouldVirtualizeFlatTree(count: number): boolean {
|
||||
return count >= 100;
|
||||
return count > 0;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -42,8 +42,9 @@ test("flattenTree preserves depth and node type for virtualized sidebar rows", (
|
|||
);
|
||||
});
|
||||
|
||||
test("shouldVirtualizeFlatTree only virtualizes very large sidebar trees", () => {
|
||||
assert.equal(shouldVirtualizeFlatTree(99), false);
|
||||
test("shouldVirtualizeFlatTree virtualizes every non-empty sidebar tree", () => {
|
||||
assert.equal(shouldVirtualizeFlatTree(0), false);
|
||||
assert.equal(shouldVirtualizeFlatTree(1), true);
|
||||
assert.equal(shouldVirtualizeFlatTree(100), true);
|
||||
});
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue