Add Telex banner and link to AI tab - #2306
Conversation
📊 Performance Test ResultsComparing 0414876 vs trunk site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change |
katinthehatsite
left a comment
There was a problem hiding this comment.
In terms of functionality, it works as expected. I don't love how it looks visually but I will approve since it does what it says it does
Would you have any thoughts on the wording? The idea is to have the CTA point to Telex while also teasing what Telex is about. |
|
Hey team @wojtekn ! What seems to strike me is the persistency of showing this. I was thinking there could be a chance to approach it in more of a 'promotional' way by showing a small banner / notice at the top. This can be combined by showing an option to 'Build with Telex' in the menu after the banner is dismissed. Some options are more on Telex brand, and one is more moderate.
|
|
@Marinatsu, thanks for the exploration. The last two look a bit too aggressive in the Studio layout; the third is a bit hidden, and the first looks like a great match. |
I agree with everything you said! |
| ); | ||
|
|
||
| const handleDismissBanner = () => { | ||
| localStorage.setItem( 'dontShowTelexBanner', 'true' ); |
There was a problem hiding this comment.
I used browser storage as it's a low-priority state - it's acceptable that it will reset when the user clears the browser storage.
| <TelexIcon /> | ||
| <span className="text-gray-900"> | ||
| { createInterpolateElement( | ||
| __( 'Build blocks with Telex. <button>Learn more</button>' ), |
There was a problem hiding this comment.
I made 'Learn more' consistent with other places.
sejas
left a comment
There was a problem hiding this comment.
Changes look good. I confirm I see the Build blocks with Telex banner on the top of the Assistant conversation. When the banner is closed by pressing the x, the banner doesn't appear anymore. I also see the Build with Telex in the assistant menu. Both links open the Telex website in the browser
I suggest adding the ↗ next to the link in the banner, similarly to other link in the app like Xdebug:
I also suggest adding the link to the word Telex ↗. I would expect "Learn more" link to open documentation and not the app itself. So the banner could look like:







Related issues
Proposed Changes
I propose adding a new Telex banner at the top of the Assistant tab and "Build with Telex" menu item in the AI input dropdown.
Testing Instructions
npm startPre-merge Checklist