[Feat] configurable-main-page-keymaps#9
Merged
ChenMiaoi merged 2 commits intoChenMiaoi:developfrom Mar 9, 2026
Merged
Conversation
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
ChenMiaoi
changed the title
Owner
Owner
|
I test this function, it works well. But I think put the sync rendering at top right may better. 
It seems too snarrow. |
|
Codex Review: Didn't find any major issues. Another round soon, please! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
- queue palette and mailbox-open sync requests onto background workers so the TUI stays responsive - close the command palette after sync commands are queued so status updates remain visible on the main screen - render footer sync progress for startup, manual, and auto-sync states so users can track background activity Signed-off-by: Chao Liu <chao.liu.zevorn@gmail.com>
- split background-sync TUI regression coverage out from the feature implementation commit - add manual-sync state-machine coverage for duplicate requests, busy workers, failures, disconnects, and completion refresh - add non-blocking mailbox-open, sync palette dismissal, and footer progress rendering coverage for startup, manual, and auto-sync states Signed-off-by: Chao Liu <chao.liu.zevorn@gmail.com>
zevorn
force-pushed
the
feat/configurable-main-page-keymaps
branch
from
eaf2a21 to
a74fee8
Compare
Contributor
Author
You are right. I changed it. |
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Feat/configurable-main-page-keymaps
Related Issues
Link issues if applicable.
What Changed
Testing
List what you ran locally.
Documentation
Checklist
Notes For Reviewers
Anything that needs special attention during review.