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

docs: ✏️ Fix broken logo image in storybook (top left) #274

Merged
merged 1 commit into from
May 21, 2021

Conversation

caleboleary
Copy link

Fix broken logo image in storybook (top left)

Closes: #262

The below structure change seems to fix it, not sure why ¯_(ツ)_/¯
Screen Shot 2021-05-21 at 2 44 04 PM

I did find some still open issues on storybook's github about theming cache issues, see storybookjs/storybook#13200

Adding the following seemed to clear it out and let me see the fixed logo. I didn't want to commit it since I don't really know what it might impact for a logo fix (low value) but something to try if you can't get theme changes to show in your local version.
Screen Shot 2021-05-21 at 2 44 24 PM

Fix broken logo image in storybook (top left)

✅ Closes: #262
@caleboleary caleboleary requested a review from a team May 21, 2021 19:50
Copy link
Contributor

@Chris-Boyle Chris-Boyle left a comment

Choose a reason for hiding this comment

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

Excellent!

@codyarose
Copy link
Contributor

From the docs:

NOTE: The flag --no-manager-cache disables the internal caching of Storybook and can serverely impact your Storybook loading time, so only use it when you need to refresh Storybook's UI, such as when editing themes.
https://storybook.js.org/docs/react/api/cli-options

So it's probably a good move not to commit that haha

@SeanGroff SeanGroff merged commit ea4859d into master May 21, 2021
@SeanGroff SeanGroff deleted the issue/262-broken-storybook-logo branch May 21, 2021 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Storybook top left logo broken image link
4 participants