Skip to content

[Feat] configurable main page keymaps#5

Merged
ChenMiaoi merged 2 commits intoChenMiaoi:developfrom
zevorn:feat/configurable-main-page-keymaps
Mar 8, 2026
Merged

[Feat] configurable main page keymaps#5
ChenMiaoi merged 2 commits intoChenMiaoi:developfrom
zevorn:feat/configurable-main-page-keymaps

Conversation

@zevorn
Copy link
Contributor

@zevorn zevorn commented Mar 7, 2026

Summary

Describe the intent of this change and the user-visible outcome.

Related Issues

Link issues if applicable.

What Changed

Testing

List what you ran locally.


Documentation

  • No documentation change needed
  • README updated
  • Config / workflow docs updated
  • Other docs updated

Checklist

  • The change is focused on one topic
  • Commit messages follow the repository convention
  • Tests were added or updated when needed
  • User-visible behavior was documented when needed
  • I checked for regressions in related workflows

Notes For Reviewers

Anything that needs special attention during review.

@ChenMiaoi ChenMiaoi changed the title Feat/configurable main page keymaps [Feat] configurable main page keymaps Mar 7, 2026
@zevorn zevorn force-pushed the feat/configurable-main-page-keymaps branch from 928b28b to 67838bc Compare March 7, 2026 15:59
@ChenMiaoi
Copy link
Owner

@codex

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 67838bc7d8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

@zevorn zevorn force-pushed the feat/configurable-main-page-keymaps branch from 67838bc to 5dda4ca Compare March 8, 2026 00:24
@ChenMiaoi
Copy link
Owner

@codex

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Swish!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

@zevorn zevorn force-pushed the feat/configurable-main-page-keymaps branch 2 times, most recently from 975d1d4 to 855c977 Compare March 8, 2026 15:14
@codecov
Copy link

codecov bot commented Mar 8, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@zevorn zevorn force-pushed the feat/configurable-main-page-keymaps branch from 855c977 to 372352b Compare March 8, 2026 15:23
@ChenMiaoi
Copy link
Owner

@codex

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 372352b170

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

zevorn added 2 commits March 8, 2026 23:40
- add UiKeymap enum (Default / Vim) to RuntimeConfig and config parser
- implement vim-style h/l focus + j/k navigation alongside default i/k + j/l
- wire keymap into input handler, render footer shortcuts, and config palette
- add config set/get support for ui_keymap with live keymap switching

Signed-off-by: Chao Liu <chao.liu.zevorn@gmail.com>
Signed-off-by: Chao Liu <chao.liu.zevorn@gmail.com>
@zevorn zevorn force-pushed the feat/configurable-main-page-keymaps branch from 372352b to 5147a5b Compare March 8, 2026 15:43
@ChenMiaoi
Copy link
Owner

LGTM!

@ChenMiaoi ChenMiaoi merged commit 0aff0cc into ChenMiaoi:develop Mar 8, 2026
4 checks passed
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