Skip to content
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

Add a basic storybook story listing all the @wordpress/icons #20266

Merged
merged 2 commits into from
Feb 18, 2020

Conversation

youknowriad
Copy link
Contributor

Capture d’écran 2020-02-17 à 1 48 42 PM

Adds a Storybook page listing all the icons with a simple input to filter the icons per name.

@youknowriad youknowriad added the [Type] Developer Documentation Documentation for developers label Feb 17, 2020
@youknowriad youknowriad self-assigned this Feb 17, 2020
<Icon icon={ paragraph } size={ 48 } />
<Icon icon={ icons.paragraph } />
<Icon icon={ icons.paragraph } size={ 36 } />
<Icon icon={ icons.paragraph } size={ 48 } />
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice dimensions, 1x, 1.5x, 2x. 👌

@youknowriad
Copy link
Contributor Author

@gziolo Storyshots is failing for a weird reason here. (hooks)

@gziolo
Copy link
Member

gziolo commented Feb 17, 2020

@gziolo Storyshots is failing for a weird reason here. (hooks)

Yes, this is a known issue with stories and their integrations with other parts of Storybook. The approach we took in other places was to move the part that handles hook to a separate component, e.g.:

const LibraryExample = () => { ... };

export const Library = () => <LibraryExample />;

:)

@youknowriad youknowriad force-pushed the add/list-icons-storybook branch from be24294 to a71ed3b Compare February 17, 2020 13:49
@youknowriad youknowriad merged commit bdb7349 into master Feb 18, 2020
@youknowriad youknowriad deleted the add/list-icons-storybook branch February 18, 2020 09:42
@github-actions github-actions bot added this to the Gutenberg 7.6 milestone Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants