Skip to content

Upgrade lightningcss to 1.30.0 #17979

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

Merged
merged 4 commits into from
May 12, 2025
Merged

Upgrade lightningcss to 1.30.0 #17979

merged 4 commits into from
May 12, 2025

Conversation

philipp-spiess
Copy link
Member

@philipp-spiess philipp-spiess commented May 12, 2025

@philipp-spiess philipp-spiess requested a review from a team as a code owner May 12, 2025 11:00
@philipp-spiess philipp-spiess changed the title Upgrade lightningcss Upgrade lightningcss to 1.20.0 May 12, 2025
@RobinMalfait RobinMalfait changed the title Upgrade lightningcss to 1.20.0 Upgrade lightningcss to 1.30.0 May 12, 2025
Copy link
Member

@RobinMalfait RobinMalfait left a comment

Choose a reason for hiding this comment

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

Just a small question, I wonder why some of the rules are split right now.

Can you also add a changelog entry?

Comment on lines +5289 to +5291
.shadow-xl {
box-shadow: var(--tw-inset-shadow), var(--tw-inset-ring-shadow), var(--tw-ring-offset-shadow), var(--tw-ring-shadow), var(--tw-shadow);
}
Copy link
Member

Choose a reason for hiding this comment

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

Interesting, what happened here 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

I think this is fixing an issue in ordering where previously the --tw-shadow could sometimes appear after the box-shadow assignment because of how @supports was resolved. This generates more code now but it's corrent in a sense that even if the supports query works, the assignment is in the right order still.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah that's fair, just an interesting change. I would be smaller again if we didn't have the @supports but this is fine (and correct) so let's merge it!

@philipp-spiess philipp-spiess merged commit 4fba87b into main May 12, 2025
7 checks passed
@philipp-spiess philipp-spiess deleted the chore/lightningcss branch May 12, 2025 13:49
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.

2 participants