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

feat(atomic): add support for at-rules, keyframes and pseudo classes #913

Merged
merged 2 commits into from Feb 6, 2022

Conversation

jpnelson
Copy link
Contributor

Add support for at-rules, keyframes and pseudo classes

Motivation

See #911

Summary

This changes the atomizer function to a different parsing method: visiting the parents of each declaration, and atomizing in a way that includes the parents.

Test plan

I tested the result on the website, and also added some unit tests.

I wonder if we need to expand the website to include more demos of some of these things? It could be useful to have a sort of proof of concept page

@Anber
Copy link
Collaborator

Anber commented Feb 6, 2022

I wonder if we need to expand the website to include more demos of some of these things?

Gitbook allowed us to use a community plan so we will migrate all documentation to it someday. If you wanna create documentation for atomic, I can configure GitBook asap.

@Anber Anber merged commit dee7fa1 into callstack:master Feb 6, 2022
@Anber
Copy link
Collaborator

Anber commented Feb 18, 2022

Hi @jpnelson
It looks like this PR broke some tests from one of your previous PRs. The strangest part is that the problem occurs only on CI. Do you have any ideas?

https://app.circleci.com/pipelines/github/callstack/linaria/671/workflows/97883cda-c473-4a5b-b276-5fec74c39011/jobs/8631

@Anber
Copy link
Collaborator

Anber commented Feb 18, 2022

It was just excess console.log :)

@jpnelson
Copy link
Contributor Author

Oh shoot, sorry I missed that! Thank you for figuring that out.

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.

None yet

2 participants