Skip to content

fix(ui): display tag labels instead of numeric IDs#18

Open
talissoncosta wants to merge 1 commit intomainfrom
fix/display-tag-labels
Open

fix(ui): display tag labels instead of numeric IDs#18
talissoncosta wants to merge 1 commit intomainfrom
fix/display-tag-labels

Conversation

@talissoncosta
Copy link
Contributor

@talissoncosta talissoncosta commented Feb 6, 2026

Summary

  • Fixes bug where tags displayed numeric IDs (e.g., 27540) instead of labels (e.g., ui, theme)
  • Adds colored tag chips matching the Flagsmith dashboard style (light tinted background, subtle border, darkened text)

Screenshots

image image

Changes

  • Add FlagsmithTag interface and getProjectTags method to FlagsmithClient
  • Update useFlagsmithProject hook to fetch tags and provide tagMap
  • Create shared TagChip component with Flagsmith-style coloring
  • Add colorUtils with hex parsing, luminance calculation, and tag chip style helpers
  • Update ExpandableRow and FeatureDetailsGrid to use TagChip
  • Update mock handlers with realistic tag data for development

Test plan

  • All existing tests pass
  • New colorUtils unit tests pass (hex parsing, contrast, edge cases)
  • Run locally with yarn start and verify tags show labels with colors

🤖 Generated with Claude Code

The Flagsmith API returns tags as an array of numeric IDs, but the UI
was displaying these IDs directly. This fix fetches tag definitions
from the project and creates a lookup map to display labels instead.

- Add FlagsmithTag interface and getProjectTags method to FlagsmithClient
- Update useFlagsmithProject hook to fetch tags and provide tagMap
- Update components to use tagMap for resolving tag labels
- Update mock handlers with realistic tag data

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@talissoncosta talissoncosta requested a review from Zaimwa9 February 6, 2026 16:20
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