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.

Sign up for GitHub

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

Jump to bottom

feat: postcss8 & tailwind2 #64

Merged
merged 7 commits into from Apr 5, 2021
Merged

feat: postcss8 & tailwind2 #64

merged 7 commits into from Apr 5, 2021

Conversation

rigor789
Copy link
Member

@rigor789 rigor789 commented Dec 3, 2020 •
edited

Migrated plugin to postCSS 8 api

Updated to tailwind 2.x

Adjusted supported/unsupported rules to remove some of the unsupported features from tw2:

  • ring utilities
  • shadow utilities (until supported)

Known issues:

  • linear-gradients don't work: nativescript does not parse the css variables tw uses to compose the gradients
  • animations don't work - nativescript fails to parse the animation shorthand property & throws
    • ✅ fixed by @thisduck by expanding shorthand properties to their individual properties.

@rigor789 rigor789 mentioned this pull request Dec 4, 2020
Closed
thisduck and others added 4 commits December 9, 2020 00:30
@thisduck
feat: expand animations for NS. (#67)
0d7b779
@thisduck
feat: support flexbox classes (#72)
503f6d9 
* (feat): Adding flexbox classes.

* (refactor): Combine plugins for export.

* (refactor): include postcss files.

* (refactor): include dependencies.

* Represent animation duration as seconds.
@rigor789
chore: clean up repo
bfb0c2b
@rigor789
chore: build & pin postcss
d6d06ba
@rigor789
Copy link
Member Author

I've pinned postcss to 8.1.13 as any higher version doesn't seem to remove rules properly with our configuration, I've opened an issue postcss/postcss#1526 to figure out if we can get it fixed.

@rigor789 rigor789 mentioned this pull request Mar 15, 2021
Closed
@rigor789 rigor789 merged commit a99e7fc into master Apr 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rigor789 @thisduck