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

Support full customization #43

Merged
merged 3 commits into from
Dec 23, 2020
Merged

Support full customization #43

merged 3 commits into from
Dec 23, 2020

Conversation

vadimdemedes
Copy link
Owner

This PR rewrites how tailwind-rn works. Previously it specified an allow list of utilities (classes), which would strip out any custom properties from your config, like custom colors. This PR changes it to block unsupported utilities and CSS properties instead, while allowing everything else.

@eduludi
Copy link
Contributor

eduludi commented Nov 24, 2020

@vadimdemedes Any idea when this is going to be merged and published? How can we help? Thanks! 😊

@vadimdemedes
Copy link
Owner Author

@eduludi Best help would be to test it in real apps and let me know whether you see any issues, like missing classes. Would appreciate it!

@eduludi
Copy link
Contributor

eduludi commented Dec 1, 2020

@vadimdemedes so far, so good 👍

@aswinmohanme
Copy link

aswinmohanme commented Dec 15, 2020

It's throwing a error: Error: Unable to resolve module './styles.json' from 'node_modules/tailwind-rn/index.js' error when I run it.

I installed the custom branch using the yarn add vadimdemedes/tailwind-rn#custom-config

@vadimdemedes
Copy link
Owner Author

@aswinmohanme I just tried it out myself and it seems to work. Did you install this PR via npm install --save-dev vadimdemedes/tailwind-rn#custom-config? Make sure to follow the instructions in the readme too.

@vadimdemedes vadimdemedes merged commit 0e8c925 into master Dec 23, 2020
@vadimdemedes vadimdemedes deleted the custom-config branch December 23, 2020 19:59
@aswinmohanme
Copy link

I can confirm it works.

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