Skip to content

docs: storybook updates for icon sets, arrow and chevron #2491

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

jawinn
Copy link
Collaborator

@jawinn jawinn commented Feb 5, 2024

Description

Related component story fixes for a fix in the Icon migration #2347 , that helps fix the wrong workflow icons appearing for arrow and chevron.

These are both icons with names that exist in both icon sets. Some component stories need the icon set specified. These changes were cherry-picked from that PR to avoid breaking changes.

How and where has this been tested?

Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.

Validation steps

  • Storybook pages for the affected components have not changed
  • No VRT changes

Regression testing

Validate:

If components have been modified, VRTs have been run on this branch:

  • VRTs have been run and looked at.
  • Any VRT changes have been accepted (by reviewer and/or PR author), or there are no changes.

To-do list

  • I have read the contribution guidelines.
  • [x I have updated relevant storybook stories and templates.
  • ✨ This pull request is ready to merge. ✨

Copy link
Contributor

github-actions bot commented Feb 5, 2024

File metrics

Summary

Total size: 3.92 MB*

🎉 No changes detected in any packages

* Size determined by adding together the size of the main file for all packages in the library.
* Results are not gzipped or minified.
* An ASCII character in UTF-8 is 8 bits or 1 byte.

Copy link
Contributor

github-actions bot commented Feb 5, 2024

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

@jawinn jawinn added the run_vrt For use on PRs looking to kick off VRT label Feb 5, 2024
@jawinn jawinn force-pushed the jawinn/css-511-stories-ui-icon-sets branch from 9ea9fec to 584cb6f Compare February 5, 2024 18:23
Related fixes for Icon migration, that helps fix the wrong workflow
icons appearing for arrow and chevron.

These are both icons with names that exist in both icon sets.
Some component stories need the icon set specified.
@jawinn jawinn force-pushed the jawinn/css-511-stories-ui-icon-sets branch from 584cb6f to 4d5e662 Compare February 5, 2024 18:26
@pfulton pfulton merged commit 4d5e662 into main Feb 5, 2024
@pfulton pfulton deleted the jawinn/css-511-stories-ui-icon-sets branch February 5, 2024 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run_vrt For use on PRs looking to kick off VRT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants