Skip to content

Rename 'Open site' to 'Open local site' in header - #2856

Merged
sejas merged 1 commit into
trunkfrom
update-open-site-label
Mar 20, 2026
Merged

Rename 'Open site' to 'Open local site' in header#2856
sejas merged 1 commit into
trunkfrom
update-open-site-label

Conversation

@shaunandrews

@shaunandrews shaunandrews commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

Related issues

  • Related to pfHvTO-16h-p2

Proposed Changes

image
  • Change "Open site" to "Open local site" in the site header to better distinguish it from remote/synced sites

Testing Instructions

  1. npm start
  2. Select any site
  3. Verify the header link now reads "Open local site ↗"

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 23911e3 vs trunk

app-size

Metric trunk 23911e3 Diff Change
App Size (Mac) 1236.59 MB 1236.59 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk 23911e3 Diff Change
load 1609 ms 1933 ms +324 ms 🔴 20.1%

site-startup

Metric trunk 23911e3 Diff Change
siteCreation 6110 ms 7120 ms +1010 ms 🔴 16.5%
siteStartup 3925 ms 3920 ms 5 ms ⚪ 0.0%

Results are median values from multiple test runs.

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

@wojtekn

wojtekn commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

I'm not sure we need to be that explicit in the local development tool, but the change seems harmless. We could consider updating other places to be consistent, though:

apps/studio/src/components/content-tab-overview.tsx
apps/studio/src/components/content-tab-import-export.tsx
apps/studio/src/ipc-handlers.ts
apps/studio/src/tests/show-site-context-menu.test.ts
@sejas
sejas merged commit e6162ae into trunk Mar 20, 2026
12 checks passed
@sejas
sejas deleted the update-open-site-label branch March 20, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants