Skip to content

Conversation

@wojtekn
Copy link
Contributor

@wojtekn wojtekn commented Dec 22, 2025

Related issues

Proposed Changes

I propose adding a new left-aligned footer link "Build blocks with Telex" in the AI assistant tab.

CleanShot 2025-12-22 at 19 01 11@2x

Testing Instructions

  1. Start the app with npm start
  2. Create or open a site
  3. Navigate to the Assistant tab
  4. Scroll to the bottom of the assistant interface
  5. Verify the footer shows two elements:
    • Left side: "Build blocks with Telex" (where "Telex" is a clickable link)
    • Right side: "Powered by experimental AI. Learn more"
  6. Click the "Telex" link and verify it opens https://telex.automattic.ai/ in your browser

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
Copy link
Contributor

📊 Performance Test Results

Comparing 0e0cb45 vs trunk

site-editor

Metric trunk 0e0cb45 Diff Change
load 13060.00 ms 17127.00 ms +4067.00 ms 🔴 31.1%

site-startup

Metric trunk 0e0cb45 Diff Change
siteCreation 23790.00 ms 24842.00 ms +1052.00 ms 🔴 4.4%
siteStartup 10033.00 ms 10035.00 ms +2.00 ms 🔴 0.0%

Results are median values from multiple test runs.

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

Copy link
Contributor

@katinthehatsite katinthehatsite left a comment

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
Copy link
Contributor Author

wojtekn commented Dec 23, 2025

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
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
Copy link

Marinatsu commented Dec 23, 2025

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
Copy link
Contributor Author

wojtekn commented Dec 23, 2025

@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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants