Skip to content

fix: change syncing icon and do not rotate pending one (syncStatus) #3205

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
juancruz-f7l wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

fix: change syncing icon and do not rotate pending one (syncStatus) #3205

juancruz-f7l wants to merge 1 commit into from

Conversation

@juancruz-f7l
Copy link
Contributor

@juancruz-f7l juancruz-f7l commented Jan 8, 2026 •
edited
Loading

Description

  • Changes syncing icon
  • Makes pending one not rotate

Specification:

image

@github-actions github-actions bot added fix react Changes affect packages/react labels Jan 8, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

✅ No New Circular Dependencies

No new circular dependencies detected. Current count: 0

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

📦 Alpha Package Version Published

Use pnpm i github:factorialco/f0#npm/alpha-pr-3205 to install the package

Use pnpm i github:factorialco/f0#5babbdfe3374fd9a2f6f6705bcb5317d9678effb to install this specific commit

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

🔍 Visual review for your branch is published 🔍

Here are the links to:

@github-actions
Copy link
Contributor

github-actions bot commented Jan 8, 2026

Coverage Report for packages/react

Status Category Percentage Covered / Total
🔵 Lines 37.46% 5772 / 15408
🔵 Statements 36.88% 5893 / 15976
🔵 Functions 26.55% 1224 / 4609
🔵 Branches 27.15% 3293 / 12127
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/react/src/components/value-display/types/syncStatus/syncStatus.tsx 28.57% 0% 0% 28.57% 72-106
Generated in workflow #10106 for commit 87e7616 by the Vitest Coverage Report Action

atd
atd approved these changes Jan 9, 2026
View reviewed changes
Copy link

@atd atd left a comment

Choose a reason for hiding this comment

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

thank you!

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

Reviewers

@atd atd atd approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

fix react Changes affect packages/react

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@juancruz-f7l @atd