Agentic UI: Rework the new-user onboarding screens - #4413
Conversation
TIL: this is nice and makes testing so much easier 👍 |
📊 Performance Test ResultsComparing 1ba27d5 vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
| viewBox="0 0 200 130" | ||
| fill="none" | ||
| xmlns="http://www.w3.org/2000/svg" | ||
| aria-hidden="true" |
There was a problem hiding this comment.
I see that in some places we specifically set it to true while in others just pass true as default. I am wondering if we should stick to one approach? It is not a big deal though, mostly a nit
| } | ||
| /> | ||
| <p className={ styles.legal }> | ||
| { __( 'By continuing, you agree to our' ) }{ ' ' } |
There was a problem hiding this comment.
Should we use createInterpolateElement with a single translatable string containing placeholder markers here to avoid splitting the sentence in 4 different bits?
| ], | ||
| }, | ||
| { | ||
| title: __( 'Build with Studio Code' ), |
There was a problem hiding this comment.
@shaunandrews what do you think about this feedback?
There was a problem hiding this comment.
The first screen explains the basics of studio, and the second emphasizes the new focus on AI and Studio Code. The second screen also serves as a second chance for us to ask for login, with the emphasis that you need it for the AI features—the aim is to get more people to login by showing them what they're missing.
katinthehatsite
left a comment
There was a problem hiding this comment.
Thanks for making the adjustments!
I am still seeing two minor things on my end that mostly happen on the smallest screens:
- the checkmark does a weird split across lines in the feature descriptions:
- I also think we can improve the spacing for the legal items here:
both of these issues happen when you resize Studio to the smallest possible screen
@shaunandrews I remember that during the demo (walkthrough) call, someone mentioned that the checkmarks felt like actual checkboxes. Or maybe I misunderstood. At least that was my first impression as well. They reminded me of the onboarding screens in apps where you see things like “Enable notifications” or “Subscribe to updates,” and you expect to be able to check or uncheck them. Maybe it’s just a personal impression. |







Related issues
How AI was used in this PR
Claude ported the design from the exploration branch (
explore-site-centric-conversation-chrome, #3975) onto current trunk, and I reviewed every screen in the running app. Worth a close look at two things the AI changed rather than copied:welcomeStylesinto connect and tour for the feature card and auth buttons. That's extracted into shared components instead — which surfaced real drift, e.g. the tour usedvariant="outline"for Log in while welcome usedsolid, under a comment claiming they matched.EmptyBackgroundis parameterized on top of trunk's version rather than taking the exploration's copy, since f61ebe7 changed that component after the branch forked.Proposed Changes
Second pass on the first-run flow: welcome → concept tour → Add a site. Each screen now owns one story instead of three screens repeating the same pitch.
Testing Instructions
The welcome only shows with zero sites, so point the app at an empty config rather than deleting yours:
Pre-merge Checklist