Skip to content

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

Merged
Anber merged 2 commits into
callstack:masterfrom
jpnelson:issue/911
Feb 6, 2022
Merged

feat(atomic): add support for at-rules, keyframes and pseudo classes#913
Anber merged 2 commits into
callstack:masterfrom
jpnelson:issue/911

Conversation

@jpnelson
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator

Anber commented Feb 18, 2022

It was just excess console.log :)

@jpnelson
Copy link
Copy Markdown
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.

2 participants