Skip to content

Add Telex banner and link to AI tab - #2306

Merged
wojtekn merged 8 commits into
trunkfrom
add/telex-link-in-assistant-tab
Dec 30, 2025
Merged

Add Telex banner and link to AI tab#2306
wojtekn merged 8 commits into
trunkfrom
add/telex-link-in-assistant-tab

Conversation

@wojtekn

@wojtekn wojtekn commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

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.

CleanShot 2025-12-30 at 09 48 39@2x

Testing Instructions

  1. Start the app with npm start
  2. Create or open a site
  3. Navigate to the Assistant tab
  4. Verify the conversation shows "Build blocks with Telex" banner
  5. Verify "Learn more" is clickable and points to Telex landing page
  6. Verify Assistant text area three-dots menu shows Telex link

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@wojtekn
wojtekn requested a review from a team December 22, 2025 18:00
@wojtekn wojtekn self-assigned this Dec 22, 2025
@github-actions

github-actions Bot commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

📊 Performance Test Results

Comparing 0414876 vs trunk

site-editor

Metric trunk 0414876 Diff Change
load 10125.00 ms 8384.00 ms -1741.00 ms 🟢 -17.2%

site-startup

Metric trunk 0414876 Diff Change
siteCreation 14205.00 ms 14379.00 ms +174.00 ms 🔴 1.2%
siteStartup 6011.00 ms 5953.00 ms -58.00 ms 🟢 -1.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change

@katinthehatsite katinthehatsite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@wojtekn

wojtekn commented Dec 23, 2025

Copy link
Copy Markdown
Contributor Author

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.

@katinthehatsite

Copy link
Copy Markdown
Contributor

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.

I think the wording itself is fine but perhaps the position could be improved. It looks a bit strange to have it in the footer opposite of another link. The Left/right split feels arbitrary and slightly unbalanced. Some options that come to mind:

  • Build blocks with Telex · Learn about experimental AI - centered in the middle, similarly, for example to GH:
Screenshot 2025-12-23 at 9 35 26 AM
  • Or left-align both, stacked vertically with subtle spacing.

I think either of those would make it visually cleaner.

@Marinatsu

Marinatsu commented Dec 23, 2025

Copy link
Copy Markdown

Hey team @wojtekn !
In terms of language, I think it's looking good.
I agree with @katinthehatsite that the positioning is a bit strange.

What seems to strike me is the persistency of showing this.
It does seem too prominent to have it below the input field at all times.

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.

telex 1 telex 2 telex 3 telex 4
@wojtekn

wojtekn commented Dec 23, 2025

Copy link
Copy Markdown
Contributor Author

@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.

@wojtekn
wojtekn marked this pull request as draft December 23, 2025 14:18
@Marinatsu

Copy link
Copy Markdown

@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!
We can combine the first and the second. After you dismiss the banner, there can still be a way to access Telex from the menu.

);

const handleDismissBanner = () => {
localStorage.setItem( 'dontShowTelexBanner', 'true' );

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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>' ),

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made 'Learn more' consistent with other places.

@wojtekn
wojtekn marked this pull request as ready for review December 29, 2025 11:22

@sejas sejas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Image

I suggest adding the next to the link in the banner, similarly to other link in the app like Xdebug:

Image

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:

Image

@gavande1 gavande1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this. LGTM 👍

Image
@wojtekn

wojtekn commented Dec 29, 2025

Copy link
Copy Markdown
Contributor Author

@sejas makes sense, I replaced link in 57e27c1

@wojtekn wojtekn changed the title Add Telex link to AI assistant footer Dec 30, 2025
@wojtekn
wojtekn merged commit 8439496 into trunk Dec 30, 2025
11 checks passed
@wojtekn
wojtekn deleted the add/telex-link-in-assistant-tab branch December 30, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants