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

Update base.html #181

Closed
wants to merge 1 commit into from
Closed

Update base.html #181

wants to merge 1 commit into from

Conversation

ishwon
Copy link
Contributor

@ishwon ishwon commented Oct 6, 2021

Do you need to load tailwind.min.css from the CDN if you're already including it in the assets? I am assuming that it was built to provide additional classes.

Do you need to load tailwind.min.css from the CDN if you're already including it in the assets? I am assuming that it was built to provide additional classes.
@Abdur-rahmaanJ
Copy link
Member

Well @ishwon i need to contact either sandeep or anyone who knows tailwind. I don't know what tree shaking both have, like we brute loaded the files, also some notus classes are not loading. So as we are not sure, we keep both until we can build it

@ishwon
Copy link
Contributor Author

ishwon commented Oct 6, 2021

Noted. I might have a look at it this weekend. Been doing some Tailwind recently, that's why it triggered my interest when I saw that. I'll see what notus does and if I can help with that.

@Abdur-rahmaanJ
Copy link
Member

@ishwon i am somewhat in the dark, great if you can sort this out!

@Abdur-rahmaanJ
Copy link
Member

Feel free to close this PR as you progress!

@ishwon
Copy link
Contributor Author

ishwon commented Oct 7, 2021

I made another PR with fix for the Tailwind duplication. Closing this.

@ishwon ishwon closed this Oct 7, 2021
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