Skip to content

fix(chat): reverse shimmer sweep to left-to-right#5650

Merged
TheodoreSpeaks merged 1 commit into
stagingfrom
fix/shimmer-direction
Jul 13, 2026
Merged

fix(chat): reverse shimmer sweep to left-to-right#5650
TheodoreSpeaks merged 1 commit into
stagingfrom
fix/shimmer-direction

Conversation

@TheodoreSpeaks

Copy link
Copy Markdown
Collaborator

Summary

  • Flipped the active-label shimmer (subagent/tool rows) to sweep left-to-right instead of right-to-left by adding reverse to the animation shorthand
  • Reduced-motion pulse fallback unaffected — it overrides the animation shorthand entirely with the symmetric shimmer-pulse

Type of Change

  • Bug fix

Testing

Tested manually. bun run lint and bun run check:api-validation:strict pass.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jul 13, 2026 11:32pm

Request Review

@cursor

cursor Bot commented Jul 13, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Single-line visual animation tweak with no logic, API, or accessibility fallback changes.

Overview
The shared text shimmer animation in shimmer-text.module.css now runs with animation-direction: reverse, so the highlight band moves left-to-right instead of right-to-left on active labels (e.g. ShimmerText, ThinkingLoader, subagent/tool rows).

The prefers-reduced-motion path is unchanged: it still replaces the sweep with the shimmer-pulse opacity fallback.

Reviewed by Cursor Bugbot for commit dbbd28d. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adjusts the shimmer text animation used in chat labels. The main changes are:

  • Adds reverse to the shimmer-sweep animation shorthand.
  • Leaves the reduced-motion shimmer-pulse override unchanged.

Confidence Score: 5/5

This looks safe to merge after a small visual animation fix.

  • The change is limited to a local CSS module.
  • The reduced-motion fallback still replaces the sweep animation.
  • The changed direction appears to run opposite the intended left-to-right sweep.

apps/sim/components/ui/shimmer-text.module.css

Important Files Changed

Filename Overview
apps/sim/components/ui/shimmer-text.module.css Updates the shimmer animation shorthand, with a likely direction mismatch against the existing keyframes.

Reviews (1): Last reviewed commit: "fix(chat): reverse shimmer sweep to left..." | Re-trigger Greptile

Comment thread apps/sim/components/ui/shimmer-text.module.css
@TheodoreSpeaks TheodoreSpeaks merged commit 7bcae15 into staging Jul 13, 2026
18 checks passed
@TheodoreSpeaks TheodoreSpeaks deleted the fix/shimmer-direction branch July 13, 2026 23:45
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.

1 participant