Skip to content

fix(Tag): IconOnly width adjustments - #5335

Merged
jeremie-lucca merged 2 commits into
release/22.1from
fix/tag.nolabel.adjustment
Sep 14, 2026
Merged

jeremie-lucca merged 2 commits into
release/22.1from
fix/tag.nolabel.adjustment

Conversation

@jeremie-lucca

Copy link
Copy Markdown
Contributor

Description

IconOnly width adjustments


Capture d’écran 2026-09-11 à 15 25 25

@jeremie-lucca jeremie-lucca added this to the 22.1 milestone Sep 11, 2026
Copilot AI balanced review requested due to automatic review settings September 11, 2026 13:31
@jeremie-lucca
jeremie-lucca requested review from a team as code owners September 11, 2026 13:31
@jeremie-lucca jeremie-lucca added the 🔖🐛 FIX Bug / Visual fix (PR should target Master branch) label Sep 11, 2026
@jeremie-lucca
jeremie-lucca changed the base branch from master to release/22.1 September 11, 2026 13:31
@c-3po c-3po Bot added the 📖 Documentation changes Requires a Prisme update label Sep 11, 2026
@github-actions

Copy link
Copy Markdown

🚀 Storybook preview deployed: https://pub-dc6d99acd6874e2aaff6219dd8a13ae2.r2.dev/PR-5335/index.html

Copilot AI 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.

🟢 Approval recommended

No unresolved review issues were identified.

Pull request overview

Updates Tag horizontal padding so icon-only and hidden-label variants render at their intended widths.

Changes:

  • Adds configurable inline padding.
  • Applies reduced padding to icon-only states.
  • Expands QA coverage across S, M, and L sizes.
File summaries
File Description
stories/qa/tags/tags.stories.html Adds size variants for icon-only and hidden-label tags.
packages/scss/src/components/tag/vars.scss Defines default inline padding.
packages/scss/src/components/tag/mods.scss Adds reduced icon-only padding.
packages/scss/src/components/tag/index.scss Detects relevant icon-only and hidden-label states.
packages/scss/src/components/tag/component.scss Uses the configurable padding variable.
Review details
  • Files reviewed: 5/5 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jeremie-lucca
jeremie-lucca enabled auto-merge (squash) September 11, 2026 13:34
@jeremie-lucca
jeremie-lucca merged commit 4309105 into release/22.1 Sep 14, 2026
10 of 11 checks passed
@jeremie-lucca
jeremie-lucca deleted the fix/tag.nolabel.adjustment branch September 14, 2026 09:31

This branch was successfully deployed

1 active deployment
staging — a0d04856 Deployed Sep 11, 2026 by jeremie-lucca via Deploy Lucca Front staging #1139
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📖 Documentation changes Requires a Prisme update 🔖🐛 FIX Bug / Visual fix (PR should target Master branch)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants