Skip to content

fix: remove duplicate 'Loading projects...' in sidebar#376

Open
EricBlanquer wants to merge 1 commit intositeboon:mainfrom
EricBlanquer:fix/duplicate-loading-projects
Open

fix: remove duplicate 'Loading projects...' in sidebar#376
EricBlanquer wants to merge 1 commit intositeboon:mainfrom
EricBlanquer:fix/duplicate-loading-projects

Conversation

@EricBlanquer
Copy link
Contributor

@EricBlanquer EricBlanquer commented Feb 10, 2026

Summary

  • Removed a duplicate <h3>Loading projects...</h3> heading that appeared twice in the sidebar during project loading
  • The duplicate was accidentally introduced in commit fea8e30 (i18n translation) which re-added a heading that had been replaced by the progress bar in commit 9e03acb
image image

Summary by CodeRabbit

  • Bug Fixes
    • Removed a duplicated heading and extra descriptive line from the Projects loading state, leaving a single loading header alongside the existing spinner and progress UI to reduce visual clutter and improve clarity during load.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 10, 2026

Walkthrough

Removed a duplicate "Loading projects" heading and an extra descriptive paragraph from the Sidebar component's Projects list loading state; spinner and progress UI remain unchanged.

Changes

Cohort / File(s) Summary
Sidebar Loading State Cleanup
src/components/Sidebar.jsx
Deleted redundant heading and paragraph in the Projects list loading UI, leaving a single loading header plus existing spinner and progress indicators.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • viper151

Poem

🐰 I nibbled code by candlelight,

A duplicate heading gave a fright,
I hopped it out with nimble paws,
Now loading reads without the flaws,
Hooray — one less line, applause! 🎉

🚥 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 'fix: remove duplicate 'Loading projects...' in sidebar' directly and clearly describes the main change in the pull request—removing a duplicate heading that appeared in the sidebar's loading UI.
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

No actionable comments were generated in the recent review. 🎉


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.

@EricBlanquer EricBlanquer force-pushed the fix/duplicate-loading-projects branch from 4e1f575 to 760f486 Compare February 10, 2026 11:15
The i18n translation commit (fea8e30) accidentally reintroduced
a second <h3> "Loading projects..." heading that was previously
replaced by the loading progress bar in commit 9e03acb.
@EricBlanquer EricBlanquer force-pushed the fix/duplicate-loading-projects branch from 760f486 to e610c5c Compare February 10, 2026 11:19
@blackmammoth
Copy link
Collaborator

thanks @EricBlanquer...I will add this in the WIP refactor PR #374

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.

2 participants