Skip to content

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 <im3dabasia1@git.wordpress.org>

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
Co-authored-by: im3dabasia <im3dabasia1@git.wordpress.org>
Co-authored-by: mirka <0mirka00@git.wordpress.org>
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 <wildworks@git.wordpress.org>
Co-authored-by: ciampo <mciampini@git.wordpress.org>
Co-authored-by: tyxla <tyxla@git.wordpress.org>
Gulamdastgir-Momin pushed a commit to Gulamdastgir-Momin/gutenberg that referenced this pull request Jan 23, 2025
Co-authored-by: im3dabasia <im3dabasia1@git.wordpress.org>
Co-authored-by: mirka <0mirka00@git.wordpress.org>
Gulamdastgir-Momin pushed a commit to Gulamdastgir-Momin/gutenberg that referenced this pull request Jan 23, 2025
…Press#68526)

* Revert "fix: Fix media query and reduced motion error (WordPress#68464)"

This reverts commit 921536e.

* Add extension explicitly in import statements

Co-authored-by: t-hamano <wildworks@git.wordpress.org>
Co-authored-by: ciampo <mciampini@git.wordpress.org>
Co-authored-by: tyxla <tyxla@git.wordpress.org>
chriszarate pushed a commit to chriszarate/gutenberg that referenced this pull request Jul 1, 2025
…Press#68526)

* Revert "fix: Fix media query and reduced motion error (WordPress#68464)"

This reverts commit 921536e.

* Add extension explicitly in import statements

Co-authored-by: t-hamano <wildworks@git.wordpress.org>
Co-authored-by: ciampo <mciampini@git.wordpress.org>
Co-authored-by: tyxla <tyxla@git.wordpress.org>
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