-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
[2.x] Refactor HydeFront styles to TailwindCSS #2024
[2.x] Refactor HydeFront styles to TailwindCSS #2024
Conversation
5a34e99
to
4e2fa27
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## new-asset-system #2024 +/- ##
====================================================
Coverage 100.00% 100.00%
Complexity 1906 1906
====================================================
Files 195 195
Lines 5078 5080 +2
====================================================
+ Hits 5078 5080 +2 ☔ View full report in Codecov by Sentry. |
4e2fa27
to
a1c530f
Compare
a1c530f
to
b9578a7
Compare
b9578a7
to
712cf95
Compare
There are going to be some tiny visual regressions/changes, so I'm going to tackle all of those later. Want to get the main refactor done first. |
…ydefront Remove the `hyde.css` file from HydeFront
… precedence" This reverts commit 16fa89c.
…tailwind-components [2.x] Extract HydeFront partials for Tailwind component templates
Abstract
We are updating how HydeFront is handled for v2. Instead of declaring styles directly in HydeFront alongside Tailwind, we will refactor those styles into Tailwind.
HydeFront will serve two main purposes:
app.css
file bundled with new apps, available through the HydeFront CDN.app.css
, which we will preconfigure.For example, users can include our Tailwind styles granularly using just the
app.css
file from the HydeFront package. This file will be compiled alongside Tailwind. If users prefer customization, they can remove the import and add the specific styles they want.Todo: Test that checks if config files are matching should test the Tailwind config
Related Changes
hyde.css
file from HydeFront #2037I think changelog in many parts can be summarized as: