Skip to content

enhance: show the first page of history while the rest is still loading - #2870

Open
bertyhell wants to merge 1 commit into
sourcegit-scm:developfrom
bertyhell:perf/paged-history-loading
Open

bertyhell wants to merge 1 commit into
sourcegit-scm:developfrom
bertyhell:perf/paged-history-loading

Conversation

@bertyhell

Copy link
Copy Markdown

On large repositories the history view stayed empty until all commits (up
to MaxHistoryCommits) were read and the graph was built. Now the first
1000 commits (with graph) are shown as soon as git log has produced them,
while the same process keeps streaming the rest, which replaces the list
when done. A small spinner next to the history options shows that more
commits are being loaded.

Swapping in the full list keeps the current selection and scroll position.
Navigating to a commit that is not loaded yet (search results, links,
parent/child navigation, etc.) shows its details immediately and selects it
in the graph once it has been loaded.

🤖 Generated with Claude Code

On large repositories the history view stayed empty until all commits (up
to `MaxHistoryCommits`) were read and the graph was built. Now the first
1000 commits (with graph) are shown as soon as `git log` has produced them,
while the same process keeps streaming the rest, which replaces the list
when done. A small spinner next to the history options shows that more
commits are being loaded.

Swapping in the full list keeps the current selection and scroll position.
Navigating to a commit that is not loaded yet (search results, links,
parent/child navigation, etc.) shows its details immediately and selects it
in the graph once it has been loaded.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KCxH1zwDXLd9kfAAkcf6PZ

This branch has not been deployed

No deployments
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