Closed tunny17 closed 3 days ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
cio-cloud | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 1, 2024 5:14am |
cio-com | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 1, 2024 5:14am |
cio-docs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 1, 2024 5:14am |
Thank you for following the naming conventions for pull request titles! 🙏
Hide Generate progress report on desktop
When the user clicks download, show a loading state but changing the text to Loading...
Disable download until all fields are filled.
@tunny17
$nodeStore
which isn't rendered on mobile. Rendering from the dom directly can lead to many inconsistencies where on different screensizes the background image can look different because the width is different - I can send images if you need better understanding.My recommendation would be that when the user clicks download, you render the container with a width of 550px
but it shouldn't be visible to the user and then make sure it can be downloaded. This way both on mobile or any desktop you can make sure that the image is consistent.
[x] Please remove the gray circle on the progress, the progress doesn't look like the preview. Instead of using a progress bar, please implement it with a html/css to match the design
[x] Facebook doesn't work
[x] The text font is not the same as in the design
What does this PR do?
Fixes #313
Type of change
How should this be tested?
Checklist
Required
pnpm build
console.logs
git pull origin main
Appreciated