Skip to content

✨ Tag component + πŸ’„ Dataviz dark mode#1190

Merged
mariush2 merged 10 commits into
mainfrom
feat/tag
Nov 28, 2025
Merged

✨ Tag component + πŸ’„ Dataviz dark mode#1190
mariush2 merged 10 commits into
mainfrom
feat/tag

Conversation

@mariush2
Copy link
Copy Markdown
Contributor

@mariush2 mariush2 commented Nov 27, 2025

Azure DevOps links

User story


  • Needs to be tested locally by reviewer

Description

  • Added darkmode variants for dataviz colors
  • Created tag component

Copilot AI review requested due to automatic review settings November 27, 2025 13:43
@mariush2 mariush2 changed the title feat/tag ✨ Tag component + πŸ’„ Dataviz dark mode Nov 27, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a new Tag molecule component for displaying labels with optional leading/trailing icons and multiple color variants. The implementation includes the component itself, utility functions for color mapping, Storybook stories for documentation, and updates to design tokens to support the new color schemes.

  • Adds Tag component with support for 8 color variants (blue, green, purple, orange, yellow, dark pink, pink, grey)
  • Implements TAG_COLORS utility for mapping color props to text/background colors
  • Extends design tokens (light/dark modes) with new color variants including berry, orange, and additional shades

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 12 comments.

Show a summary per file
File Description
src/molecules/Tag/Tag.tsx Main Tag component implementation with icon support and styled container
src/molecules/Tag/Tag.utils.ts Color mapping utility defining text and background colors for each variant
src/molecules/Tag/Tag.stories.tsx Storybook stories demonstrating Tag variants, colors, and icon configurations
src/atoms/style/lightTokens.ts Light theme token definitions for new color variants (berry, orange, additional shades)
src/atoms/style/darkTokens.ts Dark theme token definitions for new color variants with inverted color schemes
src/atoms/style/colors.ts TypeScript color constant exports mapping CSS variables to color values

πŸ’‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Comment thread src/atoms/style/darkTokens.ts
Comment thread src/atoms/style/colors.ts
Comment thread src/molecules/Tag/Tag.tsx
Comment thread src/atoms/style/darkTokens.ts
Comment thread src/atoms/style/colors.ts Outdated
Comment thread src/molecules/Tag/Tag.utils.ts
Comment thread src/molecules/Tag/Tag.tsx
Comment thread src/atoms/style/lightTokens.ts
Comment thread src/atoms/style/darkTokens.ts
Comment thread src/molecules/Tag/Tag.stories.tsx Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 27, 2025

Coverage Report

Status Category Percentage Covered / Total
πŸ”΄ Lines 99.62% (🎯 100%) 16349 / 16410
πŸ”΄ Statements 99.62% (🎯 100%) 16349 / 16410
πŸ”΄ Functions 99.65% (🎯 100%) 1172 / 1176
πŸ”΄ Branches 99.67% (🎯 100%) 4021 / 4034
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/atoms/style/colors.ts 100% 100% 100% 100%
src/atoms/style/darkTokens.ts 100% 100% 100% 100%
src/atoms/style/lightTokens.ts 100% 100% 100% 100%
src/molecules/Tag/Tag.tsx 100% 100% 100% 100%
src/molecules/Tag/Tag.utils.ts 100% 100% 100% 100%
Generated in workflow #1969 for commit 5bc0085 by the Vitest Coverage Report Action

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@mariush2 mariush2 marked this pull request as ready for review November 28, 2025 10:59
@mariush2 mariush2 requested a review from a team as a code owner November 28, 2025 10:59
@mariush2 mariush2 requested a review from aslakihle November 28, 2025 10:59
Copy link
Copy Markdown
Contributor

@aslakihle aslakihle left a comment

Choose a reason for hiding this comment

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

😎 🏷️

@mariush2 mariush2 merged commit f9662af into main Nov 28, 2025
9 of 11 checks passed
@mariush2 mariush2 deleted the feat/tag branch November 28, 2025 11:18
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.

3 participants