Skip to content

feat(ui-tag): remove the inline variant from Tag v2 - #2654

Merged
ToMESSKa merged 1 commit into
masterfrom
INSTUI-5127-remove-the-inline-variant-of-tag-from-v-2
Jul 24, 2026
Merged

feat(ui-tag): remove the inline variant from Tag v2#2654
ToMESSKa merged 1 commit into
masterfrom
INSTUI-5127-remove-the-inline-variant-of-tag-from-v-2

Conversation

@ToMESSKa

@ToMESSKa ToMESSKa commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

BREAKING CHANGE: the inline variant and the variant prop have been removed from Tag v2.

Closes: INSTUI-5127

ISSUE:

  • the inline variant of tag is no longer supported so it needs to get removed from v2

TEST PLAN:

  • check the examples on the Tag documentation page, they should work as usual

BREAKING CHANGE: the inline variant and the variant prop have been removed from Tag v2.

Refs: INSTUI-5127
@ToMESSKa
ToMESSKa force-pushed the INSTUI-5127-remove-the-inline-variant-of-tag-from-v-2 branch from 230c25e to 69c58fe Compare July 23, 2026 09:16
@ToMESSKa ToMESSKa changed the title refactor(ui-tag)!: remove the inline variant from Tag v2 feat(ui-tag): remove the inline variant from Tag v2 Jul 23, 2026
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-24 11:34 UTC

@github-actions

Copy link
Copy Markdown
Contributor

Visual regression report

⚠️ Changes detected.

Status Count
Unchanged 31
Changed 1
New 0
Removed 0

📊 View full report

Diff images (1)

metric-pill-tag-timeselect-text.png — 105269 pixels differ

Baselines come from the visual-baselines branch. They refresh on every merge to master.

github-actions Bot pushed a commit that referenced this pull request Jul 23, 2026
@ToMESSKa
ToMESSKa requested a review from matyasf July 23, 2026 10:04
Comment on lines -140 to -147
backgroundColor: componentTheme.inlineBackground,
border: `${componentTheme.inlineBorderWidth} ${componentTheme.inlineBorderStyle} ${componentTheme.inlineBorderColor}`,
borderRadius: componentTheme.inlineBorderRadius,
color: componentTheme.inlineColor,
cursor: 'text',
margin: '0 0.1875rem 0.1875rem',
...(isButton && {
'&:hover': { backgroundColor: componentTheme.inlineBackgroundHover }

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

we should ask design to remove these tokens

@joyenjoyer joyenjoyer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good

@ToMESSKa
ToMESSKa merged commit 255d84f into master Jul 24, 2026
11 of 12 checks passed
@ToMESSKa
ToMESSKa deleted the INSTUI-5127-remove-the-inline-variant-of-tag-from-v-2 branch July 24, 2026 11:34
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