Skip to content

Studio: Remove noop menu items from topbar - #4031

Merged
katinthehatsite merged 1 commit into
trunkfrom
fix/remove-noop-menu-items
Jul 1, 2026
Merged

Studio: Remove noop menu items from topbar#4031
katinthehatsite merged 1 commit into
trunkfrom
fix/remove-noop-menu-items

Conversation

@katinthehatsite

Copy link
Copy Markdown
Contributor

Related issues

Fixes STU-1954

How AI was used in this PR

It was used to update tests. Otherwise, not need as the change is simple.

Proposed Changes

This PR remove noop menu items such as "Show Tab Bar", "Show All Tabs", "Show Previous Tab", "Show Next Tab", "Move Tab to New Window", "Merge All Windows" as they are not needed with Studio being a single window application.

Testing Instructions

  • Pull the changes from this branch
  • Start Studio with npm start
  • Confirm that the topbar does not contain the elements mentioned above:
Screenshot 2026-07-01 at 3 07 07 PM

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@katinthehatsite katinthehatsite self-assigned this Jul 1, 2026
@katinthehatsite
katinthehatsite requested a review from a team July 1, 2026 13:12

@wojtekn wojtekn 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 code change looks clear.

@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 4b7f5c4 vs trunk

app-size

Metric trunk 4b7f5c4 Diff Change
App Size (Mac) 1316.83 MB 1316.83 MB 0.00 MB ⚪ 0.0%

site-editor

Metric trunk 4b7f5c4 Diff Change
load 744 ms 760 ms +16 ms ⚪ 0.0%

site-startup

Metric trunk 4b7f5c4 Diff Change
siteCreation 6510 ms 6484 ms 26 ms ⚪ 0.0%
siteStartup 1862 ms 1862 ms 0 ms ⚪ 0.0%

Results are median values from multiple test runs.

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

@katinthehatsite
katinthehatsite merged commit a3250c0 into trunk Jul 1, 2026
12 checks passed
@katinthehatsite
katinthehatsite deleted the fix/remove-noop-menu-items branch July 1, 2026 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants