-
Notifications
You must be signed in to change notification settings - Fork 51
Studio: Refresh the CTA for users without sites #2038
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: trunk
Are you sure you want to change the base?
Conversation
📊 Performance Test ResultsComparing 627bbdf vs trunk site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change |
|
@sejas I am leaving this as draft for now, just wanted to get your thoughts on design and copy for now. What do you think? |
|
Sounds good! Thanks for working in this issue. I'll give it an early review. |
|
@sejas One more question - I guess with the new design, we should now display this modal when the user clicks on either of those buttons, right?
|
|
Yes! both buttons will trigger the same modal if the user doesn't have any WPcom site. |

Related issues
Closes STU-912
Proposed Changes
This PR adds a modal with call to action for users that don't have any WP.com sites:
Original design: RToz6tIuQ7nlZrikBte4GU-fi-10114_134320 (minus the image on the top as we are not supposed to have the image there.
I made a small change to the design with the modal title:
Testing Instructions
SynctabConnect sitePre-merge Checklist