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

Fix SCSS Media Query Compilation Issue in Storybook #68464

Merged
merged 1 commit into from
Jan 2, 2025

Conversation

im3dabasia
Copy link
Contributor

@im3dabasia im3dabasia commented Jan 2, 2025

Follow up: #68419
Comment: #68419 (comment)

What?

Fixed an issue where the preprocessor was incorrectly compiling SCSS media queries in Storybook.

Why?

Replaced the previous media query syntax with a corrected version to ensure SCSS compiles properly in Storybook, resolving CI failures.

How?

Replaced the syntax: @media (prefers-reduced-motion: no-preference).

Testing Instructions

  1. Run npm run storybook:dev
  2. Observe the storybook compiles without any errors.

@im3dabasia im3dabasia marked this pull request as ready for review January 2, 2025 13:20
Copy link

github-actions bot commented Jan 2, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: im3dabasia <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@mirka mirka added [Type] Bug An existing feature does not function as intended Storybook Storybook and its stories for components [Package] Edit Site /packages/edit-site labels Jan 2, 2025
Copy link
Member

@mirka mirka left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix!

@mirka mirka enabled auto-merge (squash) January 2, 2025 13:32
@mirka mirka merged commit 921536e into WordPress:trunk Jan 2, 2025
70 of 71 checks passed
@github-actions github-actions bot added this to the Gutenberg 20.1 milestone Jan 2, 2025
@t-hamano
Copy link
Contributor

t-hamano commented Jan 3, 2025

Thank you for the fix 🙇‍♂️

t-hamano added a commit that referenced this pull request Jan 7, 2025
bph pushed a commit to bph/gutenberg that referenced this pull request Jan 8, 2025
t-hamano added a commit that referenced this pull request Jan 10, 2025
t-hamano added a commit that referenced this pull request Jan 10, 2025
* Revert "fix: Fix media query and reduced motion error (#68464)"

This reverts commit 921536e.

* Add extension explicitly in import statements

Co-authored-by: t-hamano <[email protected]>
Co-authored-by: ciampo <[email protected]>
Co-authored-by: tyxla <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Edit Site /packages/edit-site Storybook Storybook and its stories for components [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants