Skip to content

refactor(devices): drop inline synced-ago text, neutral re-sync button#3386

Merged
tofikwest merged 1 commit into
mainfrom
refactor/device-sync-button-neutral
Jul 10, 2026
Merged

refactor(devices): drop inline synced-ago text, neutral re-sync button#3386
tofikwest merged 1 commit into
mainfrom
refactor/device-sync-button-neutral

Conversation

@tofikwest

@tofikwest tofikwest commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Two small reverts on the device-sync control (per Tofik's review of the live version):

  • Removed the inline "Synced Xh ago" text — the dropdown's info block already shows last/next sync times, so it was redundant.
  • Re-sync button back to the neutral outline variant — a lone refresh icon doesn't warrant primary-color emphasis.

Layout stays as merged: right-aligned row, compact provider select, icon-only ↻ beside it (aria-label/tooltip "Sync now").

89/89 devices tests passing, typecheck clean.

🤖 Generated with Claude Code


Summary by cubic

Removed the inline “Synced Xh ago” text from the device sync control; last/next sync info now lives in the dropdown. Switched the re-sync button to a neutral outline variant while keeping the icon-only “Sync now” affordance and accessibility labels.

Written for commit bcdd3f3. Summary will update on new commits.

Review in cubic

The dropdown's info block already shows last/next sync times, so the
inline 'Synced Xh ago' text was redundant. The icon-only re-sync button
goes back to the neutral outline variant — a lone refresh icon doesn't
warrant primary emphasis.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TfDHFTwRgYxUoyyqGkHPDc
@vercel

vercel Bot commented Jul 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Jul 10, 2026 3:29am
comp-framework-editor Ready Ready Preview, Comment Jul 10, 2026 3:29am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal Skipped Skipped Jul 10, 2026 3:29am

Request Review

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@tofikwest tofikwest merged commit 358af80 into main Jul 10, 2026
8 of 9 checks passed
@tofikwest tofikwest deleted the refactor/device-sync-button-neutral branch July 10, 2026 03:28
@claudfuen

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.100.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants