feat(chat): replace thinking indicator with gooey loader#5525
Conversation
Promotes ThinkingLoader from the landing route group to the shared components/ui barrel and swaps it in for the 4-square color loader in the chat's PendingTagIndicator. Removes the now-dead animate-thinking-block keyframe/animation.
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview In workspace chat, Cleanup: deletes the unused Reviewed by Cursor Bugbot for commit fc159a1. Configure here. |
Greptile SummaryThis PR moves the gooey thinking loader into shared UI and uses it in chat. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (2): Last reviewed commit: "fix(chat): keep a stable Thinking… label..." | Re-trigger Greptile |
Greptile flagged that phase text repeatedly re-announces to assistive tech during a single pending stream; a static label avoids the repeated live-region updates while keeping the gooey shape morph.
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit fc159a1. Configure here.
Summary
ThinkingLoader(the gooey SVG loader from the landing page) fromapp/(landing)/components/thinking-loaderto the sharedcomponents/uibarrelPendingTagIndicator, replacing the 4-square color loader + "Thinking…" textanimate-thinking-blockkeyframe/animation fromtailwind.config.tsType of Change
Testing
Tested manually.
bun run type-check,bun run lint, and thespecial-tagstest suite all pass.Checklist