Skip to content

Commit 6008461

Browse files
committed
fix(core): align root tree nodes
Signed-off-by: John Yanarella <jyanarella@nvidia.com>
1 parent 7e217b8 commit 6008461

4 files changed

Lines changed: 30 additions & 4 deletions

File tree

Lines changed: 2 additions & 2 deletions
Loading

‎projects/core/.visual/tree.png‎

Lines changed: 2 additions & 2 deletions
Loading

‎projects/core/src/tree/tree-node.test.ts‎

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,23 @@ describe(TreeNode.metadata.tag, () => {
132132
);
133133
});
134134

135+
it('should align leaf root nodes with expandable root nodes',async()=>{
136+
constleafHeader=element.shadowRoot!.querySelector<HTMLElement>('[part="_node-header"]')!;
137+
constexpandableHeader=nestedNodeElement.shadowRoot!.querySelector<HTMLElement>('[part="_node-header"]')!;
138+
139+
expect(leafHeader.getBoundingClientRect().x).toBe(expandableHeader.getBoundingClientRect().x);
140+
});
141+
142+
it('should align multi-select leaf root nodes with expandable root nodes',async()=>{
143+
tree.selectable='multi';
144+
awaitPromise.all([elementIsStable(tree),elementIsStable(element),elementIsStable(nestedNodeElement)]);
145+
146+
constleafHeader=element.shadowRoot!.querySelector<HTMLElement>('[part="_node-header"]')!;
147+
constexpandableHeader=nestedNodeElement.shadowRoot!.querySelector<HTMLElement>('[part="_node-header"]')!;
148+
149+
expect(leafHeader.getBoundingClientRect().x).toBe(expandableHeader.getBoundingClientRect().x);
150+
});
151+
135152
it('should show icon button if expandable',async()=>{
136153
expect(element.shadowRoot.querySelector('nve-icon-button')).toBeFalsy();
137154

‎projects/core/src/tree/tree.global.css‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,3 +26,12 @@ nve-tree-node > nve-tree-node:state(is-expandable):state(selectable-multi) {
2626
nve-tree-node:has(nve-tree-node:state(is-expandable))::part(_nodes) {
2727
padding-inline-start:var(--nve-ref-size-400);
2828
}
29+
30+
nve-tree:has(>nve-tree-node:state(is-expandable)) >nve-tree-node:not(:state(is-expandable))::part(_node) {
31+
padding-inline-start:var(--nve-ref-size-400);
32+
}
33+
34+
nve-tree:has(>nve-tree-node:state(is-expandable):state(selectable-multi))
35+
>nve-tree-node:not(:state(is-expandable)):state(selectable-multi)::part(_node) {
36+
padding-inline-start:var(--nve-ref-size-500);
37+
}

0 commit comments

Comments
 (0)