Skip to content

[Website] Remove obsolete Site Manager UI scaffolding - #4078

Merged
adamziel merged 1 commit into
trunkfrom
remove-obsolete-site-manager-ui
Jul 16, 2026
Merged

[Website] Remove obsolete Site Manager UI scaffolding#4078
adamziel merged 1 commit into
trunkfrom
remove-obsolete-site-manager-ui

Conversation

@adamziel

Copy link
Copy Markdown
Collaborator

This removes Site Manager UI code that no production import path can reach. Keeping it around makes the Dock code look broader than it is and leaves dead full-screen overlay pieces next to the one overlay helper the website still uses.

The branch deletes the old blueprints panel, temporary-site notice, and storage-type components. It also removes the stale TemporarySiteNotice test mock. The full-screen Overlay, OverlayHeader, OverlayBody, PlaygroundLogo, and overlayStyles exports are gone, while OverlaySection stays because SavedPlaygroundsPanel still imports it.

This intentionally does not rename SiteManager, change Dock behavior, alter the open-by-default experiment, or touch the public overlay query parameter.

Verified with:

source "$HOME/.nvm/nvm.sh" && nvm use
git diff --check origin/trunk...HEAD
! git grep -n -E 'BlueprintsPanel|TemporarySiteNotice|PlaygroundLogo|Overlay(Header|Body)|overlayStyles' -- packages/playground/website
! git grep -n 'site-manager/storage-type' -- packages/playground/website
npm exec nx lint playground-website
npm exec nx typecheck playground-website
npm exec nx test playground-website

I also opened the local website and checked the New Playground and Playgrounds panes at desktop and narrow viewport sizes. The pane structure, spacing, empty/current states, and gallery cards matched the expected Dock layouts. The deleted components were not mounted.

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Removes unreachable/obsolete Site Manager UI components and legacy full-screen overlay scaffolding, leaving only OverlaySection for the website’s remaining overlay usage.

Changes:

  • Deleted BlueprintsPanel, TemporarySiteNotice, and StorageType components (and their CSS modules).
  • Simplified components/overlay to only export OverlaySection and trimmed overlay CSS accordingly.
  • Removed the stale TemporarySiteNotice test mock.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/playground/website/src/components/site-manager/temporary-site-notice/style.module.css Deletes CSS for removed TemporarySiteNotice.
packages/playground/website/src/components/site-manager/temporary-site-notice/index.tsx Deletes removed TemporarySiteNotice component implementation.
packages/playground/website/src/components/site-manager/storage-type/style.module.css Deletes CSS for removed StorageType UI.
packages/playground/website/src/components/site-manager/storage-type/index.tsx Deletes removed StorageType component implementation.
packages/playground/website/src/components/site-manager/site-info-panel/site-tool-panels.spec.tsx Removes mock for deleted TemporarySiteNotice.
packages/playground/website/src/components/site-manager/blueprints-panel/style.module.css Deletes CSS for removed BlueprintsPanel.
packages/playground/website/src/components/site-manager/blueprints-panel/index.tsx Deletes obsolete BlueprintsPanel implementation.
packages/playground/website/src/components/overlay/style.module.css Removes full-screen overlay styles, keeps only section-related styles.
packages/playground/website/src/components/overlay/index.tsx Removes full Overlay* exports; keeps OverlaySection.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/playground/website/src/components/overlay/style.module.css
@adamziel
adamziel merged commit 72e3158 into trunk Jul 16, 2026
53 checks passed
@adamziel
adamziel deleted the remove-obsolete-site-manager-ui branch July 16, 2026 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment