Skip to content

Add none keyword #327

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 10 commits into from
Mar 11, 2023
Merged

Add none keyword #327

merged 10 commits into from
Mar 11, 2023

Conversation

tiaanl
Copy link
Collaborator

@tiaanl tiaanl commented Feb 26, 2023

For each color component, we allow the "none" keyword. The keyword will be kept in place for serialization. The "none" keyword is not allowed in the legacy syntax.

rgb and hsl are handled special, because they support the legacy color syntax. They also cannot be combined, because rgb cares about the type of the components, and hsl does not.

@bors-servo
Copy link
Contributor

☔ The latest upstream changes (presumably #328) made this pull request unmergeable. Please resolve the merge conflicts.

@tiaanl tiaanl force-pushed the add-none-keyword branch from 5383a42 to e91919c Compare March 9, 2023 11:58
@tiaanl tiaanl force-pushed the add-none-keyword branch from e647027 to 9bdf3bc Compare March 10, 2023 13:06
@tiaanl tiaanl force-pushed the add-none-keyword branch from 02295d9 to 4582f86 Compare March 10, 2023 14:37
Copy link
Member

@emilio emilio left a comment

Choose a reason for hiding this comment

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

I only looked at the new changes but seems reasonable. r=me with the bits below addressed.

@emilio
Copy link
Member

emilio commented Mar 10, 2023

@bors-servo delegate+

(@tiaanl this allows you to at-bors-servo r=emilio when comments are addressed)

@bors-servo
Copy link
Contributor

✌️ @tiaanl can now approve this pull request

@tiaanl
Copy link
Collaborator Author

tiaanl commented Mar 11, 2023

@bors-servo r=emilio

@bors-servo
Copy link
Contributor

📌 Commit da5ea89 has been approved by emilio

@bors-servo
Copy link
Contributor

⌛ Testing commit da5ea89 with merge 45bc47e...

@bors-servo
Copy link
Contributor

☀️ Test successful - checks-github
Approved by: emilio
Pushing 45bc47e to master...

1 similar comment
@bors-servo
Copy link
Contributor

☀️ Test successful - checks-github
Approved by: emilio
Pushing 45bc47e to master...

@bors-servo bors-servo merged commit 45bc47e into servo:master Mar 11, 2023
@tiaanl tiaanl deleted the add-none-keyword branch March 11, 2023 08:22
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