Skip to content

[Website] Tweak confirmations copy and placement. And preserve the Playground URL through settings update. - #4150

Merged
adamziel merged 5 commits into
trunkfrom
adamziel/fellyph-tibi-dock-followups
Jul 22, 2026
Merged

[Website] Tweak confirmations copy and placement. And preserve the Playground URL through settings update.#4150
adamziel merged 5 commits into
trunkfrom
adamziel/fellyph-tibi-dock-followups

Conversation

@adamziel

@adamziel adamziel commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Polishes four rough edges in the Dock.

The delete confirmation now names the destructive action in the button: "Delete Playground". It does not explain where the app goes after deletion; the app should just pick the next sensible Playground.

Running a Blueprint from the editor now posts a Dock-level success toast. That matters on narrow screens because the editor closes before the user can see inline feedback.

Saved Playground menus keep a disabled "Saved in browser storage" or "Saved in a local directory" row, so the menu still says why the save action is gone.

Runtime setting changes now reopen the WordPress page the user was on after PHP or networking restarts. The URL is kept only in transient Redux state. Before restoring it, the browser URL parser checks that it still belongs to the same Playground, so query strings such as ?slashes=/// survive while protocol-relative or backslash-normalized external URLs are ignored.

Testing

Delete a Playground and check that the confirm button says "Delete Playground". Run a Blueprint from the editor on a narrow viewport and check for a success toast. Open a saved Playground row menu and check that it says where the Playground is saved. Open a wp-admin page, change the PHP version, and check that the same page reopens after the runtime reload.

@adamziel
adamziel requested review from a team, Copilot and mho22 July 21, 2026 20:03

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.

Clarifies several Dock-related UX flows (delete/run/save/runtime reload) by improving notifications, preserving context across runtime reboots, and making saved-location info explicit in menus.

Changes:

  • Add “success” status to Dock operation notices and auto-dismiss success toasts.
  • Persist and restore the current WordPress URL after runtime settings changes/reboot.
  • Improve saved playground row menus and delete-modal messaging to better explain outcomes.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
packages/playground/website/src/lib/state/redux/slice-ui.ts Adds status to Dock operation notice model to support success toasts.
packages/playground/website/src/lib/state/redux/slice-sites.ts Persists a URL to restore after runtime settings changes.
packages/playground/website/src/lib/state/redux/site-management-api-middleware.ts Captures current URL before runtime reboots and stores it in site metadata.
packages/playground/website/src/lib/state/redux/boot-site-client.ts Restores stored URL after reboot, then clears the restore flag.
packages/playground/website/src/lib/state/redux/boot-site-client.spec.ts Adds coverage for reopening the previous page after runtime reboot.
packages/playground/website/src/components/saved-playgrounds-panel/index.tsx Keeps a disabled “saved location” item in row menus.
packages/playground/website/src/components/dock/style.module.css Adds success styling for Dock operation toast.
packages/playground/website/src/components/dock/dock.tsx Auto-dismisses success notices; adjusts ARIA roles/labels; adds success styling hook.
packages/playground/website/src/components/delete-site-modal/index.tsx Adds info notice explaining what happens after deletion (esp. active tab).
packages/playground/website/src/components/blueprint-editor/BlueprintBundleEditor.tsx Shows a Dock-level success toast when running a Blueprint.

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

Comment thread packages/playground/website/src/lib/state/redux/boot-site-client.ts Outdated
Comment thread packages/playground/website/src/lib/state/redux/site-management-api-middleware.ts Outdated
Comment thread packages/playground/website/src/components/saved-playgrounds-panel/index.tsx Outdated
Comment thread packages/playground/website/src/components/delete-site-modal/index.tsx Outdated
Comment thread packages/playground/website/src/components/dock/dock.tsx
@adamziel
adamziel force-pushed the adamziel/fellyph-tibi-dock-followups branch from 87572ba to 40e6cdb Compare July 21, 2026 22:41
@adamziel
adamziel force-pushed the adamziel/fellyph-tibi-dock-followups branch from 40e6cdb to f5c417f Compare July 21, 2026 23:04
@adamziel adamziel changed the title [Website] Clarify Dock destructive, run, save, and runtime flows Jul 21, 2026
@adamziel adamziel changed the title [Website] Make Dock actions explicit and restore pages after runtime reloads Jul 21, 2026
@adamziel adamziel changed the title [Website] Tweak confirmations copy and placement. And preserve the Playground URL over settings update. Jul 21, 2026
@adamziel
adamziel merged commit c33210f into trunk Jul 22, 2026
53 checks passed
@adamziel
adamziel deleted the adamziel/fellyph-tibi-dock-followups branch July 22, 2026 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment