Skip to content

Comments

Studio: Add time display over pull and push completed notification#2653

Merged
katinthehatsite merged 1 commit intotrunkfrom
add/information-for-pull-push-complete
Feb 24, 2026
Merged

Studio: Add time display over pull and push completed notification#2653
katinthehatsite merged 1 commit intotrunkfrom
add/information-for-pull-push-complete

Conversation

@katinthehatsite
Copy link
Contributor

Related issues

Proposed Changes

This PR adds time display over Push completed and Pull completed as Tooltip so that the user knows when it was completed. This works similarly to the tooltip over Push and Pull buttons:

Screenshot 2026-02-24 at 12 13 38 PM Screenshot 2026-02-24 at 12 10 44 PM

Testing Instructions

  • Pull the changes from this branch
  • Navigate to the Sync tab and connect a site
  • Once connected, start the Push process
  • Observe that once push is completed, you can see the tooltip with when the site was last pushed over Push completed text
  • Start the Pull process
  • Observe that you can see time notification over Pull completed text

Pre-merge Checklist

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

@katinthehatsite katinthehatsite self-assigned this Feb 24, 2026
@katinthehatsite katinthehatsite requested a review from a team February 24, 2026 11:20
@wpmobilebot
Copy link
Collaborator

📊 Performance Test Results

Comparing efcfb1c vs trunk

site-editor

Metric trunk efcfb1c Diff Change
load 1419.00 ms 1472.00 ms +53.00 ms 🔴 3.7%

site-startup

Metric trunk efcfb1c Diff Change
siteCreation 7075.00 ms 7074.00 ms -1.00 ms ⚪ 0.0%
siteStartup 3938.00 ms 3939.00 ms +1.00 ms ⚪ 0.0%

Results are median values from multiple test runs.

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

Copy link
Member

@sejas sejas 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 the tooltip to the pull, push success message 👍

I confirm the tooltip correctly appears when moving the mouse over the Push/Pull complete.

Image

Copy link
Contributor

@ivan-ottinger ivan-ottinger left a comment

Choose a reason for hiding this comment

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

Nice! Looks good on RTL as well:

Image

@katinthehatsite katinthehatsite merged commit e235ccf into trunk Feb 24, 2026
9 checks passed
@katinthehatsite katinthehatsite deleted the add/information-for-pull-push-complete branch February 24, 2026 13:05
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.

4 participants