Skip to content

Studio Settings: Reorder tabs and align description styling with MCP tab#2914

Merged
sejas merged 6 commits intotrunkfrom
update/studio-skills-settings
Mar 26, 2026
Merged

Studio Settings: Reorder tabs and align description styling with MCP tab#2914
sejas merged 6 commits intotrunkfrom
update/studio-skills-settings

Conversation

@sejas
Copy link
Copy Markdown
Member

@sejas sejas commented Mar 25, 2026

Related issues

Proposed Changes

  • Update Skills tab description to use a8c-body-small class for left-aligned text matching the MCP tab typography
  • Reorder tab sentence to lead with the actionable instruction
  • Move Skills tab after Account tab to group agent-related tabs (Skills, MCP) together

Testing Instructions

  • Start the app with ENABLE_AGENT_SUITE=true npm start
  • Open Settings (Cmd+,)
  • Observe the tabs in the following order: General, Account, Skills, MCP
  • Navigate to the Skills tab and verify the description text is left-aligned and uses the same typography as the MCP tab
Screenshot 2026-03-26 at 09 47 20
global-studio-settings.mp4

(Video outdated. Now it doesn't have the installed badge)

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@sejas sejas self-assigned this Mar 25, 2026
@wpmobilebot
Copy link
Copy Markdown
Collaborator

wpmobilebot commented Mar 25, 2026

📊 Performance Test Results

Comparing 1dca862 vs trunk

app-size

Metric trunk 1dca862 Diff Change
App Size (Mac) 1247.25 MB 1247.24 MB 0.01 MB ⚪ 0.0%

site-editor

Metric trunk 1dca862 Diff Change
load 1874 ms 1859 ms 15 ms ⚪ 0.0%

site-startup

Metric trunk 1dca862 Diff Change
siteCreation 8166 ms 8179 ms +13 ms ⚪ 0.0%
siteStartup 3940 ms 3929 ms 11 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@sejas sejas changed the title Skills Settings: Align description styling with MCP tab Studio Settings: Reorder tabs and align description styling with MCP tab Mar 25, 2026
@sejas sejas requested review from a team and shaunandrews March 25, 2026 22:44
<p className="a8c-body-small m-0">
{ __(
'Agents can decide to use skills to help them accomplish specialized tasks. Select the skills that will be placed in all existing and new sites.'
'Select the skills that will be placed in all existing and new sites. Agents can decide to use skills to help them accomplish specialized tasks.'
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.

Are we also planning to add the Learn more here or it won't be necessary?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good point! I added it here e3e832a

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

The changes look good to me. I left one generic question but it is not a blocker

@sejas
Copy link
Copy Markdown
Member Author

sejas commented Mar 26, 2026

I also removed the installed badge 82b4680

@sejas sejas merged commit 8218af0 into trunk Mar 26, 2026
10 of 11 checks passed
@sejas sejas deleted the update/studio-skills-settings branch March 26, 2026 10:19
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.

3 participants