Skip to content

feat(web): add skeleton to LatestCommitInfo while loading#1195

Merged
brendan-kellam merged 2 commits into
mainfrom
brendan-kellam/latest-commit-skeleton
May 12, 2026
Merged

feat(web): add skeleton to LatestCommitInfo while loading#1195
brendan-kellam merged 2 commits into
mainfrom
brendan-kellam/latest-commit-skeleton

Conversation

@brendan-kellam
Copy link
Copy Markdown
Contributor

@brendan-kellam brendan-kellam commented May 12, 2026

Summary

  • Renders a placeholder skeleton (avatar circle, author name, commit message, date) in LatestCommitInfo while the latest-commit query is pending, instead of the component being absent from the layout.

Test plan

  • Open a repo/path in the code browser and confirm the skeleton appears briefly before the real commit info loads.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 12, 2026

Walkthrough

The PR adds a loading skeleton to the latest commit info component in the code browser. While commits data is being fetched, the component renders placeholder UI elements matching the expected layout instead of showing empty state or waiting for data.

Changes

Pending State Skeleton Loaders

Layer / File(s) Summary
Pending state skeleton loading UI
packages/web/src/app/(app)/browse/components/latestCommitInfo.tsx, CHANGELOG.md
LatestCommitInfo extracts isPending from useQuery and imports Skeleton component, then returns early during pending state with placeholder avatar, message, and date shapes. CHANGELOG documents the new loading skeleton feature.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding a skeleton loading UI to the LatestCommitInfo component while data is loading.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch brendan-kellam/latest-commit-skeleton

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.

@github-actions

This comment has been minimized.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@brendan-kellam brendan-kellam merged commit 99b9af5 into main May 12, 2026
7 of 8 checks passed
@brendan-kellam brendan-kellam deleted the brendan-kellam/latest-commit-skeleton branch May 12, 2026 04:01
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