Add mobile loading skeletons and empty states#138
Conversation
|
Hi , the mobile checks pass locally:
The failing Vercel check appears to be due to deployment authorization, not code changes. Please review when possible. |
|
Please join our Discord community from the README section so we can communicate properly and provide valid labels for your contributions, which will help you score points in GSSoC. |
|
Hi @Harxhit, as of now there are no conflicts in the code. Only the PR approval and merge are pending. Please check it once. |
Your PR is quite large, so please give us some time to properly review all the changes. |
|
Hi @jamunatg2006-sys , thanks for the contribution, could you please add small demo video, walking us through the UI and feature on you local, I'm here referencing feature developed by you. I'm at edge to merge the PR, once will have that. Thanks! |
Description
This PR improves the mobile app loading and empty-state experience by adding reusable skeleton/empty-state patterns across screens that previously used abrupt loading transitions, spinner-only loading, or plain text placeholders.
Closes #119
Changes Made
Verification
corepack pnpm --filter @devcard/mobile exec tsc --noEmitcorepack pnpm --filter @devcard/mobile exec jest --runInBandcorepack pnpm --filter @devcard/mobile lintgit diff --checkScreenshots
Not added, because changes are state-based and verified locally through code checks/tests.