Update sync tooltip to use plan-agnostic language#2988
Conversation
📊 Performance Test ResultsComparing dcd96c5 vs trunk app-size
site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff) |
| <div className="a8c-body-small text-frame-text-secondary shrink-0 text-right"> | ||
| <Tooltip | ||
| text={ __( 'Sync support is available only with Business plan and above' ) } | ||
| text={ __( 'Sync support is available on selected plans only' ) } |
There was a problem hiding this comment.
I like the idea of avoiding the plan specific language but it would be nice to point the users to what plans are actually available for sync. I am wondering if we could add something like Learn more link to the tooltip where we have this mentioned in the documentation. Although perhaps, it is not the best approach to add a link to the tooltip.
It is not a blocker, I am mostly adding this as an observation
There was a problem hiding this comment.
This is a tooltip appearing on a link - the user can click the link and see plans that support sync.
Separately from this issue, we are adding Studio Sync to the pricing grids to make it clearer. Also, with the server-side change happening in parallel, we will enable Studio sync for all current paid plans.
katinthehatsite
left a comment
katinthehatsite
left a comment
There was a problem hiding this comment.
The changes look good, I left one minor comment but it is not a blocker
4 participants
Related issues
How AI was used in this PR
Claude Code was used to locate and update the tooltip string.
Proposed Changes
I proposed updating the sync tooltip text from "Sync support is available only with Business plan and above" to "Sync support is available on selected plans only".
We are adding support for Sync to other plans, which is handled on the API side, so it would be great to avoid plan-related logic on the Studio side.
Testing Instructions
Pre-merge Checklist