Fix badly aligned tree items

[alexr00]

Fixes #279421

[vs-code-engineering]

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@bpasero

Matched files:

• src/vs/workbench/browser/parts/views/treeView.ts

[Copilot]

Pull request overview

This PR attempts to fix an alignment issue with tree items (issue #279421) by restricting when the no-twisty CSS class is applied to tree items. The change limits the class to only be managed for root-level items (items whose parent is the Root instance).

• The fix adds a condition to only add/remove the no-twisty class for direct children of the root node

[Copilot]

The no-twisty class is not being removed from non-root items when templates are reused. When a template previously used for a root-level leaf node (with no-twisty class) is reused for a non-root item, the class will persist incorrectly.

The fix should ensure the class is removed for non-root items:

if (node.parent instanceof Root) {
	if (node.collapsibleState === TreeItemCollapsibleState.None) {
		templateData.container.classList.add('no-twisty');
	} else {
		templateData.container.classList.remove('no-twisty');
	}
} else {
	templateData.container.classList.remove('no-twisty');
}

Suggested change

	}
	}
	} else {
	templateData.container.classList.remove('no-twisty');