Skip to content

Conversation

bcotrim
Copy link
Contributor

@bcotrim bcotrim commented Oct 9, 2025

Related issues

Proposed Changes

  • Add cancel button UI to sync operations
  • Push can be canceled during backup creation
  • Pull can be cancelled while remote backup creation and download
Cancel button Cancelled
image image

Testing Instructions

  • Start a push/pull operation from Studio to WordPress.com
  • During the sync progress, click the X icon to the right of the progress bar
  • Verify the operation cancels
  • Test cancellation at different stages of the sync process to ensure it works during safe stages
  • Confirm cancellation is disabled after safe stages

Pre-merge Checklist

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

@bcotrim bcotrim self-assigned this Oct 9, 2025
@bcotrim bcotrim marked this pull request as draft October 9, 2025 12:53
@bcotrim bcotrim force-pushed the stu-156-studio-ability-to-cancel-the-push-and-pull branch from 2464123 to 61b5ec7 Compare October 16, 2025 22:13
@bcotrim bcotrim force-pushed the stu-156-studio-ability-to-cancel-the-push-and-pull branch from 61b5ec7 to 0c15689 Compare October 16, 2025 22:36
@bcotrim bcotrim marked this pull request as ready for review October 16, 2025 22:38
Copy link

📊 Performance Test Results

Comparing 6240411 vs trunk

site-editor

Metric trunk 6240411 Diff Change
load 12316.00 ms 10420.00 ms -1896.00 ms 🟢 -15.4%

site-startup

Metric trunk 6240411 Diff Change
siteCreation 15111.00 ms 14994.00 ms -117.00 ms 🟢 -0.8%
siteStartup 4929.00 ms 5917.00 ms +988.00 ms 🔴 20.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change

@bcotrim bcotrim requested a review from a team October 17, 2025 16:36
Copy link
Contributor

@katinthehatsite katinthehatsite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this PR, it is nice to have an ability to cancel sync operations ❤️

I have noticed two issue while testing:

  • If you cancel push first and then start pull and cancel it as well, you end up with a weird state:
Image

I think we need to update it to only include that text in relation to the exact operation that was cancelled.

  • Cancel pull was slightly overlapping the side of the screen for me:
Image

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.

2 participants