Skip to content

Conversation

Hypnosphi
Copy link
Contributor

@Hypnosphi Hypnosphi commented Sep 12, 2025

RG-2649 Update Label component

Includes changes from #8880

@Hypnosphi Hypnosphi enabled auto-merge (rebase) September 12, 2025 13:09
Copy link
Contributor

@Copilot 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

Updates the Tag component with new tagType prop and styling variants, replacing deprecated custom className styling. The changes implement a comprehensive tag system with multiple visual states including default, main, success, warning, error, and purple variants, along with an outline option.

  • Adds TagType enum with six variants (DEFAULT, MAIN, SUCCESS, WARNING, ERROR, PURPLE) and outline styling option
  • Replaces hardcoded ban label styling in UserCard with proper TagType.ERROR usage
  • Updates component styling with CSS custom properties for better theming support

Reviewed Changes

Copilot reviewed 37 out of 37 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/tag/tag.tsx Adds TagType enum, tagType and outline props, removes iconClassName prop
src/tag/tag.css Implements comprehensive styling system with CSS custom properties for all tag variants
src/user-card/card.tsx Updates ban label to use TagType.ERROR instead of custom className
src/user-card/user-card.css Removes deprecated banLabel styling
src/tag/tag.stories.tsx Updates demo to showcase all tag variants and outline options
src/global/variables.css Updates warning color value
src/global/variables_dark.css Updates grey container light color for dark theme
design-tokens.json Updates design token values for warning and grey colors
.storybook/preview.ts Removes deprecated style references

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Hypnosphi Hypnosphi marked this pull request as draft September 12, 2025 16:17
auto-merge was automatically disabled September 12, 2025 16:17

Pull request was converted to draft

@Hypnosphi Hypnosphi requested review from andrey-skl and removed request for skoch13 September 15, 2025 13:05
@Hypnosphi Hypnosphi marked this pull request as ready for review September 15, 2025 13:05
@Hypnosphi Hypnosphi merged commit b992ccb into master Sep 15, 2025
3 checks passed
@Hypnosphi Hypnosphi deleted the tag-outline branch September 15, 2025 13:59
@Hypnosphi Hypnosphi mentioned this pull request Sep 15, 2025
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