Skip to content

feat(build-tab): show tool names under icons in the Build palette#426

Open
ActiveInAI wants to merge 1 commit into
pascalorg:mainfrom
ActiveInAI:contrib/build-tab-tool-labels
Open

feat(build-tab): show tool names under icons in the Build palette#426
ActiveInAI wants to merge 1 commit into
pascalorg:mainfrom
ActiveInAI:contrib/build-tab-tool-labels

Conversation

@ActiveInAI

Copy link
Copy Markdown

What

Show the tool name under each icon in the standalone editor's Build palette — the structure tools, the Roof Features sub‑grid, and the MEP sub‑grid — instead of surfacing the name on hover only.

Why

The furniture Items panel already renders a label under every thumbnail, while the Build palette was icon‑only with a hover tooltip. That inconsistency makes the two catalogs feel different, and the build tools are harder to scan at a glance (especially for less familiar icons like the MEP set).

Change

  • Each tile becomes icon + label (flex‑col), mirroring the Items panel.
  • Grid columns widened 56px → 72px to fit the names.
  • Tooltips left in place (harmless; still works on hover).
  • Applied consistently to all three Build grids (main / Roof Features / MEP).

Notes

This is a UX/consistency proposal — happy to adjust (e.g. gate it behind a setting, or only label the main grid) if the maintainers prefer to keep the compact icon‑only style as the default. Verified in apps/editor dev: labels render, layout is stable, no behavior change.

The standalone editor's Build palette (structure tools, Roof features, and
MEP) showed icons only, surfacing the name on hover. The furniture/Items
panel already shows a name under each thumbnail, so the two catalogs felt
inconsistent and the build tools were harder to scan at a glance.

Render each tile as icon + label (matching the Items panel), widen the
grid columns (56→72px) to fit the names, and keep the tooltip as-is.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant