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

multiple storybook fixes and general enhancements #5991

Merged
merged 7 commits into from
Oct 13, 2023

Conversation

ovflowd
Copy link
Member

@ovflowd ovflowd commented Oct 12, 2023

Description

This PR does several minor adjustments to the Storybook experience, described as below:

  • fixes Storybook being unable to load absolute imports, which actually seemed to be caused (root cause) of a lack of baseUrl on our tsconfig.json file
  • fixes lack of color-scheme: dark when our theme is dark and the same for light.
    • This is originally provided by next-themes but since we don't use it within Storybook, as we simulate themes, we need to also simulate these styles.
  • Moved the previewBody directly to main.ts
  • Optimised and cleaned up the Storybook configuration

Validation

Storybook should still build and work as expected.

@ovflowd ovflowd requested a review from a team as a code owner October 12, 2023 21:22
@vercel
Copy link

vercel bot commented Oct 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 12, 2023 9:26pm

@github-actions
Copy link

github-actions bot commented Oct 12, 2023

Unit Test Coverage Report

Lines Statements Branches Functions
Coverage: 95%
94.68% (267/282) 78.12% (50/64) 92.98% (53/57)

Unit Test Report

Tests Skipped Failures Errors Time
27 0 💤 0 ❌ 0 🔥 5.6s ⏱️

@bmuenzenmeyer
Copy link
Collaborator

merging - this is a minor and i'll consider it fast-tracked

@bmuenzenmeyer bmuenzenmeyer added this pull request to the merge queue Oct 13, 2023
Merged via the queue into main with commit 1c557f7 Oct 13, 2023
14 checks passed
@bmuenzenmeyer bmuenzenmeyer deleted the chore/normalise-storybook-config branch October 13, 2023 01:12
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.

3 participants