Skip to content

Remove fake progress bar from ChatView#269

Merged
subsy merged 1 commit intomainfrom
remove-fake-progress-bar
Feb 3, 2026
Merged

Remove fake progress bar from ChatView#269
subsy merged 1 commit intomainfrom
remove-fake-progress-bar

Conversation

@subsy
Copy link
Copy Markdown
Owner

@subsy subsy commented Feb 3, 2026 •
edited by coderabbitai bot
Loading

Summary

Removes the AnimatedProgressBar component that showed a fake percentage asymptotically approaching 99%.

Why remove it:

  • Modern users know AI tasks have indeterminate timing
  • The spinner alone communicates "working" effectively
  • Fake progress bars feel deceptive to savvy users
  • Industry standard (ChatGPT, Claude.ai, Copilot) uses simple spinners, not fake percentages

Before: ⠋ ▓▓▓▓▓▓▓▓░░░░░░░░░░░░ 39% Generating PRD...

After: ⠋ Generating PRD...

Test plan

  • Verify PRD wizard shows spinner without progress bar when generating

Summary by CodeRabbit

  • Bug Fixes
    • Removed inaccurate progress indicator from loading state. The loading interface now displays only a spinner and status text for improved clarity and user experience.

Remove fake progress bar from ChatView
3a3a1d6 
The AnimatedProgressBar showed a fake percentage that asymptotically
approached 99% - a UX pattern from the 90s/2000s that modern users
see through instantly.

For AI tasks with indeterminate timing, the spinner alone is sufficient.
This matches what ChatGPT, Claude.ai, and other AI interfaces do.

-33 lines of unnecessary complexity.
@vercel
Copy link
Copy Markdown

vercel bot commented Feb 3, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
ralph-tui Ignored Ignored Feb 3, 2026 11:07pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 3, 2026 •
edited
Loading

Walkthrough

The AnimatedProgressBar component has been removed from ChatView.tsx along with its usages in the loading UI. The loading state now displays only a spinner and text, without a progress indicator.

Changes

Cohort / File(s) Summary
UI Component Removal
src/tui/components/ChatView.tsx		 Removed AnimatedProgressBar component declaration and its two usages in loading area and hint bar; loading UI now displays spinner and text only.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: removal of the fake progress bar component from ChatView, which is the primary purpose of this pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch remove-fake-progress-bar

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@subsy subsy merged commit dda82e5 into main Feb 3, 2026
9 checks passed
sakaman pushed a commit to sakaman/ralph-tui that referenced this pull request Feb 15, 2026
@subsy
Merge pull request subsy#269 from subsy/remove-fake-progress-bar
6872526 
Remove fake progress bar from ChatView
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@subsy