search

storybookjs / storybook

Storybook is the industry standard workshop for building, documenting, and testing UI components in isolation
https://storybook.js.org
MIT License
84.82k stars 9.34k forks source link

Custom icon in the tree #10379

Open thany opened 4 years ago

thany commented 4 years ago

Is your feature request related to a problem? Please describe. I'm making a MDX docs-only story that will act as the homepage of our Storybook instance, and it would be nice to give that item a homepage-worthy icon in the tree.

Describe the solution you'd like Perhaps in the <Meta> MDX element, add a prop that accepts an SVG path and a colour, that will be passed on to the tree.

Describe alternatives you've considered I don't think a workaround is possible, because icons are hardcoded in ListItem.tsx. I've tried to add an emoji into the title of a story, but that doesn't look quite right.

Are you able to assist bring the feature to reality? Maybe. Depends on the solution we're heading for, should this feature request be approved.

Additional context Not just homepages, you know. It might be helpful to easily distinguish different component by different icons, or different colours of icons.

stale[bot] commented 4 years ago

Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 30 days. Thanks!

stale[bot] commented 4 years ago

Hey there, it's me again! I am going close this issue to help our maintainers focus on the current development roadmap instead. If the issue mentioned is still a concern, please open a new ticket and mention this old one. Cheers and thanks for using Storybook!

masterbd commented 3 years ago

Can we reopen this issue?

shilman commented 3 years ago

Cc @tooppaaa @ghengeveld @domyen

tooppaaa commented 3 years ago

Would this solves your request ?

13121

thany commented 3 years ago

@tooppaaa Looks alright. A slightly different thing than what I imagined, but it would do the job (when approved). Too bad that PR introduces several things at once, and "prefix" is not the term I would use personally.