fix(tree): wrap static icon elements with tree-node-icon spacing class

[ggdouglas]

Summary

Tree nodes lost the bp6-tree-node-icon spacing class when icon was a JSX element, because <Icon> returns elements as-is. Wrap element icons in a <span class="bp6-tree-node-icon"> so their spacing matches string icons.

This was discovered during research on #8065

Code

import { Tree } from "@blueprintjs/core";

// dynamic
<Tree contents={[{ id: 0, icon: "folder-close", label: "Documents" }]} />

import { Tree } from "@blueprintjs/core";
import { FolderClose } from "@blueprintjs/icons";

// static
<Tree contents={[{ id: 0, icon: <FolderClose />, label: "Documents" }]} />

Screenshots

Before	After

Diff

[changelog-app]

Generate changelog in packages/core/changelog/@unreleased

Type (Select exactly one)

• Feature (Adding new functionality)
• Improvement (Improving existing functionality)
• Fix (Fixing an issue with existing functionality)
• Break (Creating a new major version by breaking public APIs)
• Deprecation (Removing functionality in a non-breaking way)
• Migration (Automatically moving data/functionality to a new system)

Description

fix(tree): wrap static icon elements with tree-node-icon spacing class

Check the box to generate changelog(s)

• Generate changelog entry

[changelog-app]

✅ Successfully generated changelog entry!

Need to regenerate?

Simply interact with the changelog bot comment again to regenerate these entries.

---

📋Changelog Preview

💡 Improvements

• fix(tree): wrap static icon elements with tree-node-icon spacing class (#8072)

[blueprint-previews]

fix(tree): wrap static icon elements with tree-node-icon spacing class

Build artifact links for this commit: documentation | landing | table | demo | storybook

This is an automated comment from the deploy-preview CircleCI job.