Skip to content

Agentic UI: Add a Learn more modal for publishing and syncing - #4448

Closed
shaunandrews wants to merge 1 commit into
stu-2162-single-sync-buttonfrom
stu-2162-share-dialog
Closed

Agentic UI: Add a Learn more modal for publishing and syncing#4448
shaunandrews wants to merge 1 commit into
stu-2162-single-sync-buttonfrom
stu-2162-share-dialog

Conversation

@shaunandrews

@shaunandrews shaunandrews commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Related issues

Stack

⚠️ Visual change: needs human review in light + dark mode.

Repurposed PR: previously "Share dialog", now the Learn-more modal. Preview sharing already ships as the Share menu in #4445, so this PR is just the explanatory modal.

How AI was used in this PR

Wrote the modal copy and wiring, and verified it in the local agentic UI (studio ui).

Proposed Changes

Add a single Learn more modal that explains publishing and syncing, reachable from both the Sync menu and the Publish picker.

  • Covers how publishing connects a live WordPress.com/Pressable site, how push/pull sync moves work between local and live, the destination-database-replacement warning, and eligibility.
  • One shared InfoDialog primitive backs it, so the modal stays visually consistent.

Additive on top of #4447.

Screenshots

Learn more modal "Learn more" in the Publish picker
Publishing and syncing modal Learn more item in the publish picker

Testing Instructions

  1. Open the Publish picker (unconnected site) and click Learn more — the modal opens.
  2. Open the Sync menu (connected site) and confirm Learn more opens the same modal.
  3. Confirm the copy reads correctly and Got it dismisses it.
  4. Check light + dark appearance and the browser console for errors.

Verification completed

  • npm run typecheck (clean)
  • apps/ui site-toolbar unit tests pass

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@shaunandrews

Copy link
Copy Markdown
Contributor Author

🤖 AI-written review note — verified against the current PR head.

Share remains enabled while Push/Pull is running and preview publishing reports into the same per-site sync-activity entry. In the cumulative stack, #4451 also gives preview and sync the same sync-${siteId} toast ID. Concurrent Share + Push/Pull can therefore overwrite each other’s activity/toast state and may operate on the same local site simultaneously. Please coordinate or uniquely identify these operations.

A single info modal, reachable from the Sync and Publish menus, covering how
publishing connects a live site and how push/pull sync works.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@shaunandrews
shaunandrews force-pushed the stu-2162-share-dialog branch from 1fc8bc1 to 905de69 Compare August 19, 2026 19:51
@shaunandrews shaunandrews changed the title Bring back preview links as a Share dialog in the header Agentic UI: Add a Learn more modal for publishing and syncing Aug 19, 2026
@shaunandrews

Copy link
Copy Markdown
Contributor Author

Superseded by the re-split STU-2162 stack, rebuilt on current trunk with clearer branch names and standalone-reviewable slices: #4611#4612#4613#4614#4615#4616. Closing to avoid the stale branch names here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant