Automattic
Replace empty screen with add site flow
Related issues
- Fixes STU-1063
Proposed Changes
- Remove the empty screen and replace it with add site options
Testing Instructions
- Remove all your sites from Studio
- Observe the new screen
- Continue the flow to create any of the sites
- Confirm the site was correctly created
| Before | After |
|---|---|
Pre-merge Checklist
- [ ] Have you checked for TypeScript, React or other console errors?
📊 Performance Test Results
Comparing 10fa31e1ef448706e94aa6c61ea6a97ed5d615b4 vs trunk
site-editor
| Metric | trunk | 10fa31e1ef448706e94aa6c61ea6a97ed5d615b4 | Diff | Change |
|---|---|---|---|---|
| load | 5265.00 ms | 7670.00 ms | +2405.00 ms | 🔴 45.7% |
site-startup
| Metric | trunk | 10fa31e1ef448706e94aa6c61ea6a97ed5d615b4 | Diff | Change |
|---|---|---|---|---|
| siteCreation | 14179.00 ms | 14227.00 ms | +48.00 ms | 🔴 0.3% |
| siteStartup | 5974.00 ms | 5981.00 ms | +7.00 ms | 🔴 0.1% |
Results are median values from multiple test runs.
Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
@gavande1 , thanks for the early review. Could you take another look.
The whats new model appeared
I created a new issue STU-1066 so any decision won't block from merging this PR.
The actions were not clickable
This is fixed! Could you test it again?
The changes look good 👍 , I just have one minor suggestion about disabling the
Add sitebutton in the top navigation
I created an issue to tackle that as a follow-up. STU-1068