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

Fix loading style #1073

Merged
merged 6 commits into from
Jul 30, 2024
Merged

Fix loading style #1073

merged 6 commits into from
Jul 30, 2024

Conversation

hanbyul-here
Copy link
Collaborator

@hanbyul-here hanbyul-here commented Jul 29, 2024

The style in index.html was getting ripped off on the instance level. I am not sure why this does not happen on ui level 🤔 But opening the fix since the current state is a blocker.

Please look at this pr: NASA-IMPACT/veda-config#424 to see how it looks like on the instance level.

Copy link

netlify bot commented Jul 29, 2024

Deploy Preview for veda-ui ready!

Name Link
🔨 Latest commit 5f21b87
🔍 Latest deploy log https://app.netlify.com/sites/veda-ui/deploys/66a8d471e10455000756ffce
😎 Deploy Preview https://deploy-preview-1073--veda-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@dzole0311
Copy link
Collaborator

I debugged a bit, and everything points to the path being the issue (as it was @hanbyul-here's first impression). The rest of the CSS purging worked fine on the instance side probably because of the safeList config which purged all the classes which did not match the ones added to it and not because of the content array (which contains the file globs). The content array was throwing No files found from the passed PurgeCSS option 'content' in the deployment logs for the instances, but not in the veda-ui deployments. After making the paths in the PostCSS config also dynamic , the errors seem to be now gone from the deployment logs.

@hanbyul-here hanbyul-here marked this pull request as ready for review July 30, 2024 12:49
@hanbyul-here
Copy link
Collaborator Author

Thank you so much for debugging further 🙇 @dzole0311 !! I can't review this since I am the pr creator, can either of you give an approval? @dzole0311 @sandrahoang686 ?

@hanbyul-here hanbyul-here merged commit 6b96579 into main Jul 30, 2024
12 checks passed
@hanbyul-here hanbyul-here deleted the styled-loading-page branch July 30, 2024 12:53
sandrahoang686 added a commit that referenced this pull request Jul 31, 2024
**HOLD until we merge #1073

## 🎉 Features
* Added Announcement component using USWDS Design System and adding
purge-css #1031

## 🚀 Improvements
* Add logic for tracking out-of-view Timeline playheads for E&A page
#1067
* Abstract Away Faux Modules from MapBlock Component and expose as
library component #1046

## 🐛 Fixes
- 🦗
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