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

[2.x] Remove the hyde.css file from HydeFront #2037

Conversation

caendesilva
Copy link
Member

Removes the hyde.css file from HydeFront as all its styles has now been refactored to Tailwind in #2024. This also massively simplifies the complex build steps. HydeFront now only contains the compiled app.css file to serve through the CDN. And technically we may even consider changing this to load from the file in hydephp/hyde, depending on if step two in #2024 reduces the need for this package.

@caendesilva caendesilva added the run-visual-tests Internal label to trigger Browserstack visual testing label Nov 17, 2024
Copy link

codecov bot commented Nov 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (b238280) to head (a12b565).
Report is 17 commits behind head on refactor-hydefront-styles-to-tailwind.

Additional details and impacted files
@@                            Coverage Diff                            @@
##             refactor-hydefront-styles-to-tailwind     #2037   +/-   ##
=========================================================================
  Coverage                                   100.00%   100.00%           
  Complexity                                    1906      1906           
=========================================================================
  Files                                          195       195           
  Lines                                         5080      5080           
=========================================================================
  Hits                                          5080      5080           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@caendesilva caendesilva force-pushed the remove-the-hyde-css-file-from-hydefront branch from f1cf6dd to 2034b01 Compare November 17, 2024 15:14
@caendesilva caendesilva merged commit 3c42843 into refactor-hydefront-styles-to-tailwind Nov 17, 2024
8 checks passed
@caendesilva caendesilva deleted the remove-the-hyde-css-file-from-hydefront branch November 17, 2024 15:18
@github-actions github-actions bot removed the run-visual-tests Internal label to trigger Browserstack visual testing label Nov 17, 2024
@caendesilva caendesilva changed the title Remove the hyde.css file from HydeFront [2.x] Remove the hyde.css file from HydeFront Dec 1, 2024
@caendesilva caendesilva mentioned this pull request Dec 10, 2024
99 tasks
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.

1 participant