Display the plans modal in the pull an existing site creation flow - #2197
Conversation
katinthehatsite
left a comment
There was a problem hiding this comment.
I think in terms of functionality it works fine but I am finding it strange that we display the modal in front of sort of "another modal" with the site search. To me, it would make sense to directly display the information from the NoWpcomSites modal on the screen without having the modal popup when the user logs in. I feel like this would make the design more coherent.
What do you think?
If this needs to be urgently merged before the release, we can do that and then do a follow up to improve the design as an alternative.
|
@katinthehatsite , thanks for the review and for bring it up your concern. I agree that two modals seem a bit strange, although that was the design of @Marinatsu. See p58i-ofU-p2
|
📊 Performance Test ResultsComparing 0b08bf9 vs trunk site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change |
|
I created a follow up issue to adjust the background here: https://linear.app/a8c/issue/STU-1113/studio-improve-plans-modal-in-the-pull-during-existing-site-creation |

Related issues
Proposed Changes
studioSiteIdwhen running the pull an existing site creation flow.Testing Instructions
npm startshow-modal-empty-state.mp4
find-a-perfect-plan-when-pulling-an-existing-site.mp4
Pre-merge Checklist