Skip to content

Add Installed badge to fix WordPress Skills styling consistency - #2876

Merged
sejas merged 3 commits into
trunkfrom
fix-wordpress-skills-installed-badge
Mar 24, 2026
Merged

Add Installed badge to fix WordPress Skills styling consistency#2876
sejas merged 3 commits into
trunkfrom
fix-wordpress-skills-installed-badge

Conversation

@sejas

@sejas sejas commented Mar 23, 2026

Copy link
Copy Markdown
Member

Related issues

Proposed Changes

Create a shared component to re-use styles of installed label for Skills and instructions

Testing Instructions

  1. Start the app with npm start
  2. Open AI Settings modal (site settings → AI settings)
  3. Ensure both Agent Instructions and WordPress Skills have "Installed" badges visible
  4. Toggle dark mode and verify the badges look identical across both sections
Before After
before installed-after

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors? (Tests pass, no errors)
…ons badges

Use consistent Tailwind classes (text-green-900, bg-green-50, dark:!text-green-300,
dark:bg-green-950) and add dark:fill-green-300 to the check icon, matching the
Agent Instructions section badges.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sejas sejas self-assigned this Mar 23, 2026
@wpmobilebot

wpmobilebot commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing ccf7f5b vs trunk

app-size

Metric trunk ccf7f5b Diff Change
App Size (Mac) 1278.66 MB 1278.66 MB 0.00 MB ⚪ 0.0%

site-editor

Metric trunk ccf7f5b Diff Change
load 1884 ms 1626 ms 258 ms 🟢 -13.7%

site-startup

Metric trunk ccf7f5b Diff Change
siteCreation 6054 ms 6082 ms +28 ms ⚪ 0.0%
siteStartup 3939 ms 3933 ms 6 ms ⚪ 0.0%

Results are median values from multiple test runs.

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

Replaces three duplicate inline badge implementations with a single
InstalledBadge component used in Agent Instructions, WordPress Skills
(AI settings modal), and the Skills settings tab.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sejas sejas changed the title Fix WordPress Skills Installed badge styling consistency Mar 23, 2026
@sejas
sejas requested review from a team, epeicher and katinthehatsite March 24, 2026 12:35

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

The changes make sense, thanks for making the adjustment 👍

@sejas
sejas merged commit 8007e48 into trunk Mar 24, 2026
10 checks passed
@sejas
sejas deleted the fix-wordpress-skills-installed-badge branch March 24, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants