Skip to content

feat: Only fetch latest head on switch, handle terminal focus#108

Merged
Cretezy merged 1 commit intomainfrom
push-lwpnpllwkkxz
Mar 2, 2025
Merged

feat: Only fetch latest head on switch, handle terminal focus#108
Cretezy merged 1 commit intomainfrom
push-lwpnpllwkkxz

Conversation

@Cretezy
Copy link
Owner

@Cretezy Cretezy commented Mar 2, 2025

Looking for testers for this. Seems to be working as expected for me

@Cretezy Cretezy merged commit 9f6f611 into main Mar 2, 2025
10 checks passed
@Cretezy Cretezy deleted the push-lwpnpllwkkxz branch March 2, 2025 18:27
@istudyatuni
Copy link
Collaborator

It's very slow for me now. Tested in lazyjj repo

@Cretezy
Copy link
Owner Author

Cretezy commented Mar 2, 2025

@istudyatuni Can you check if src/app.rs:252 is being hit (Event::FocusGained) too often for you? I suspect it's being called too often. Or check LogTab.focus

@istudyatuni
Copy link
Collaborator

Most time is spent in get_commit_show
image

@Cretezy
Copy link
Owner Author

Cretezy commented Mar 2, 2025

@istudyatuni Can you try #114 ?

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