Skip to content

Give a tile's badge an icon, and say what it is - #204

Merged
djthorpe merged 2 commits into
mainfrom
djt/1112/tile-badge-icon
Nov 12, 2026
Merged

djthorpe merged 2 commits into
mainfrom
djt/1112/tile-badge-icon

Conversation

@djthorpe

@djthorpe djthorpe commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

A tile's badge, such as a database's owner, didn't say what it was.

  • badgeIcon: a rendered icon, such as Carbon's 16px user icon from iconLoader, shown at 12px inside the badge, before its text. Carbon's small tag has no icon slot of its own.
  • badgeTitle: what the badge is, such as "Owner". It shows when the badge is hovered, and screen readers read it before the badge's text.
  • Shrinking: the badge now shrinks before the tile's name, so a long owner, such as pg_database_owner, is cut short instead of the name.

… the tile's name

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings November 12, 2026 19:14
@djthorpe djthorpe self-assigned this Nov 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The unconditional 70% width cap unnecessarily truncates names on badge-free tiles, leaving 30% unused instead of preserving the full available label width.

1 open issue
What changed in this PR

Adds optional icons and descriptive titles to tile badges, making information such as database ownership easier to identify in the list view.

Changes:

  • Adds badgeIcon and badgeTitle, with 12px icons, hover titles, and screen-reader text.
  • Adjusts name and badge sizing to prioritize tile names over long badge text.
  • Documents the new badge options in the frontend README.
File Description
frontend/​src/​list/​tiles.ts Adds badge fields, rendering, accessibility text, and layout styles.
frontend/​README.md Documents optional badge icons and titles.

Review effort: Balanced

Comment thread frontend/src/list/tiles.ts Resolved
…label's width

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@djthorpe
djthorpe merged commit 4fc6c69 into main Nov 12, 2026
2 checks passed
@djthorpe
djthorpe deleted the djt/1112/tile-badge-icon branch November 12, 2026 19:22
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.

2 participants