Skip to content

chore(swatch): updates the add swatch icon to use the ui rather than workflow icon#3871

Merged
cdransf merged 1 commit into
spectrum-twofrom
cdransf/s2-swatch-icon-update
May 30, 2025
Merged

chore(swatch): updates the add swatch icon to use the ui rather than workflow icon#3871
cdransf merged 1 commit into
spectrum-twofrom
cdransf/s2-swatch-icon-update

Conversation

@cdransf
Copy link
Copy Markdown
Member

@cdransf cdransf commented May 29, 2025

Description

When the swatch component was originally migrated we only had the workflow add icon. This updates the component to use the ui icon now that the latter is available.

To-do list

  • I have read the contribution guidelines.
  • I have updated relevant storybook stories and templates.
  • I have tested these changes in Windows High Contrast mode.
  • If my change impacts other components, I have tested to make sure they don't break.
  • If my change impacts documentation, I have updated the documentation accordingly.
  • ✨ This pull request is ready to merge. ✨

@cdransf cdransf self-assigned this May 29, 2025
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 29, 2025

🦋 Changeset detected

Latest commit: 64a261d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@spectrum-css/swatch Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Copy Markdown
Collaborator

@rise-erpelding rise-erpelding left a comment

Choose a reason for hiding this comment

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

Correct me if I'm wrong! I think this just needs a tiny sizing adjustment for XS!

Comment thread components/swatch/stories/template.js
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2025

🚀 Deployed on https://pr-3871--spectrum-css.netlify.app

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2025

File metrics

Summary

Total size: 1.42 MB*

🎉 No changes detected in any packages

* Size is the sum of all main files for packages in the library.
* An ASCII character in UTF-8 is 8 bits or 1 byte.

@cdransf cdransf force-pushed the cdransf/s2-swatch-icon-update branch from abd75ac to 64a261d Compare May 29, 2025 21:45
@cdransf cdransf requested a review from rise-erpelding May 29, 2025 21:45
@cdransf cdransf added size-2 S ~6-18hrs; not hard or time consuming, one or two work days to complete. skip_vrt Add to a PR to skip running VRT (but still pass the action) ready-for-review labels May 29, 2025
@cdransf cdransf requested a review from aramos-adobe May 29, 2025 22:17
@cdransf cdransf merged commit 7578ad2 into spectrum-two May 30, 2025
17 of 24 checks passed
@cdransf cdransf deleted the cdransf/s2-swatch-icon-update branch May 30, 2025 15:44
@castastrophe castastrophe mentioned this pull request Aug 6, 2025
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review size-2 S ~6-18hrs; not hard or time consuming, one or two work days to complete. skip_vrt Add to a PR to skip running VRT (but still pass the action)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants