[Enhancement] Improve Storybook Icon documentation to display all icons #346
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of Change
Used the Storybook built in IconGallery and IconItem components paired with an iterating function to build out a grid of all our icons and insert it into the Docs page for the Icon component story section.
It was decided against, but I committed a prototype method of coercing the icon gallery into being a story with this commit, modeled on this comment from a discussion of the error I was encountering.
Screenshots/Video
Screen.Recording.2024-06-04.at.3.55.15.PM.mov
Testing
Validated appeared in Web as expected. Minor bug with (5) long icon names being cut off, but it is a known bug with Storybook and Jessica thought it was acceptable.
Doc pages do not presently appear in iOS/Android, but spot checked nothing seemed amiss.
PR Checklist
Code reviewer validation:
changelog
label applied if it's to be included in the changelogPublish
Changes do not warrant a new version per the versioning guidelines as they are purely Storybook/documentation.