wp-calypso
wp-calypso copied to clipboard
Automattic
Add loading step on LIB flow
Proposed Changes
After the link-in-bio setup step, when submitting the form, the user should see the loading step that is basically a loader while the data is being handled/submitted.
It should look like the designs besides the background that is not the one on the design, it will be changed later.
Testing Instructions
- Run yarn start from the calypso root folder
- Navigate to http://calypso.localhost:3000/setup/linkInBioSetup?flow=link-in-bio&siteSlug=YOUR-SITE.wordpress.com
- Fill the form and click on continue
- You should see the loader as the described above. ( It will be fast given we have little data and the whole action happens quite fast )
- After that you should be redirect to temporary page,
http://calypso.localhost:3000/page/YOUR-SITE.wordpress.com/home. ( will be changed later )
Related to #66396 Fixes #66396
Jetpack Cloud live (direct link)
|
|
https://calypso.live?image=registry.a8c.com/calypso/app:build-39554&env=jetpack |
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Here is how your PR affects size of JS and CSS bundles shipped to the user's browser:
App Entrypoints (~151 bytes added 📈 [gzipped])
name parsed_size gzip_size
entry-stepper +408 B (+0.0%) +151 B (+0.0%)
Common code that is always downloaded and parsed every time the app is loaded, no matter which route is used.
Legend
What is parsed and gzip size?
Parsed Size: Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory. Gzip Size: Compressed size of the JS and CSS files. This much data needs to be downloaded over network.
Generated by performance advisor bot at iscalypsofastyet.com.
Hi @escapemanuele, Thank you for checking it, indeed it is a bit misaligned, pushing the fix soon. I also noticed that the print you provided is not up to date with the correct text and progress bar.
it should look like these
Thank you!
@heavyweight I appreciate your review. Yes, we will use this background color for now as @alshakero suggested.