Skip to content

refactor: Icon with appearance#674

Draft
Robbert wants to merge 1 commit intomainfrom
refactor/icon-appearance-v2
Draft

refactor: Icon with appearance#674
Robbert wants to merge 1 commit intomainfrom
refactor/icon-appearance-v2

Conversation

@Robbert
Copy link
Member

@Robbert Robbert commented Oct 23, 2025

Replace the size prop with appearance. "size" only set size whereas "appearance" also sets a value for --nl-icon-inset-block-start that center aligns icons with text inline.

Add dummy SVG icons, circle, square, and diamond that are "edge-to-edge" icons as well as "inset" versions that are inset by 2 in their viewBoxes.

Replace all stories that used "size" with their "appearance" counterparts.

Replace the size prop with appearance. "size" only set size whereas
"appearance" also sets a value for `--nl-icon-inset-block-start` that
center aligns icons with text inline.

Add dummy SVG icons, circle, square, and diamond that are "edge-to-edge"
icons as well as "inset" versions that are inset by 2 in their
viewBoxes.

Replace all stories that used "size" with their "appearance"
counterparts.
@Robbert Robbert requested a review from a team as a code owner October 23, 2025 13:25
@vercel
Copy link

vercel bot commented Oct 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
candidate Ready Ready Preview Comment Oct 23, 2025 1:26pm
candidate-storybook-non-conforming Ready Ready Preview Comment Oct 23, 2025 1:26pm
candidate-storybook-test Ready Ready Preview Comment Oct 23, 2025 1:26pm
evil-storybook Ready Ready Preview Comment Oct 23, 2025 1:26pm

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
53.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (cb1b289) to head (a970189).
⚠️ Report is 35 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #674   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        24    +2     
  Lines          240       255   +15     
  Branches        39        40    +1     
=========================================
+ Hits           240       255   +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Robbert Robbert marked this pull request as draft October 23, 2025 13:42
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