Skip to content

fix: unitless zero for <angle> values are allowed in CSS gradients#1479

Merged
ai merged 1 commit intomainfrom
fix-unitless-zero
Sep 20, 2022
Merged

fix: unitless zero for <angle> values are allowed in CSS gradients#1479
ai merged 1 commit intomainfrom
fix-unitless-zero

Conversation

@yisibl
Copy link
Copy Markdown
Member

@yisibl yisibl commented Sep 20, 2022

Problems found during lightningcss discussion: parcel-bundler/lightningcss#298

@yisibl yisibl requested a review from ai September 20, 2022 12:34
background: -webkit-linear-gradient(bottom, green, blue);
background: -o-linear-gradient(bottom, green, blue);
background: linear-gradient(0grad, green, blue);
background-image: -webkit-repeating-linear-gradient(bottom, blue, red 33.3%);
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I don't know why the test failed here, the single test(test.only('changes angle in gradient') was passed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It was prefixes cleaning test which failed.

Seems like we forgot to think about cleaning repeating gradients when we added their support.

Here is my fix 542bd85

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Nice

@ai ai merged commit 2d286dd into main Sep 20, 2022
@ai
Copy link
Copy Markdown
Member

ai commented Sep 20, 2022

Thanks. Released in 10.4.12.

@ai ai deleted the fix-unitless-zero branch September 20, 2022 23:02
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