fix(chat): sweep the text shimmer left-to-right#5926
Conversation
#5671 rewrote the shimmer-sweep keyframes to run 100% -> -100%, which is already left-to-right, but left the `reverse` from #5650 on the animation shorthand. The two flips cancel into a right-to-left sweep on every ShimmerText consumer: subagent labels, tool-call rows, and the new agent-stream thinking chrome. Drop `reverse` so direction lives in exactly one place — the keyframes.
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview The Reviewed by Cursor Bugbot for commit 16879cc. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryFixes the shimmer animation direction by removing the stale Confidence Score: 5/5The PR appears safe to merge with no actionable issues identified. The changed shorthand now plays the existing left-to-right keyframes normally, while reduced-motion behavior and all consumers remain unaffected. Important Files Changed
Reviews (1): Last reviewed commit: "fix(chat): sweep the text shimmer left-t..." | Re-trigger Greptile |
|
Blacksmith runners detected OOM events on the following jobs:
|
Summary
reversefrom theshimmer-sweepanimation inshimmer-text.module.css— the sweep runs right-to-left on stagingreverseto flip the old keyframes (implicit0%->200%, natively right-to-left) into a left-to-right sweep100%->-100%, which is already left-to-right, but leftreversein place — the two cancel out@keyframesblock 10 lines apartShimmerTextconsumer: subagent labels and tool-call rows in chat, plus the agent-stream thinking chromeType of Change
Testing
Tested manually.
bun run lint:checkandbun run check:api-validation:strictpass.Checklist