Skip to content

Conversation

@ThomasK33
Copy link
Member

Problem

The j and k keys were mapped inversely to standard Vim keybindings:

Key Before Expected (Vim)
j navigateToPrevUnviewedFile (up) Move down (next)
k navigateToNextUnviewedFile (down) Move up (prev)

Fix

Swapped the function calls so that:

  • j calls navigateToNextUnviewedFile() (down/forward)
  • k calls navigateToPrevUnviewedFile() (up/backward)

This aligns with standard Vim keybindings where j moves down and k moves up.

Change-Id: I1fdbc38047c1036a8783c99f64119890b1aed61e
Signed-off-by: Thomas Kosiewski <tk@coder.com>
@vercel
Copy link

vercel bot commented Dec 3, 2025

@ThomasK33 must be a member of the Coder team on Vercel to deploy.
- Click here to add @ThomasK33 to the team.
- If you initiated this build, request access.

Learn more about collaboration on Vercel and other options here.

@ThomasK33 ThomasK33 requested a review from kylecarbs December 3, 2025 09:24
Change-Id: I3b4839ad380eeb6362da75009ddfe018af49d201
Signed-off-by: Thomas Kosiewski <tk@coder.com>
- pr-overview.tsx: Convert undefined to null for CommentBox body prop,
  and normalize requested_reviewers null to undefined for MergeSection
- github.tsx: Replace octokit.paginate with octokit.request (paginate
  not available in @octokit/core), remove non-existent clearPending call
- user-hover-card.tsx: Add type assertion for pronouns field (not in
  @octokit/openapi-types schema yet)

Change-Id: Ib140d36609368c8920416c9c0627726c0efb4020
Signed-off-by: Thomas Kosiewski <tk@coder.com>
Change-Id: Ifebfd59cce5c6a07854adffa9e24f9ace8ca5214
Signed-off-by: Thomas Kosiewski <tk@coder.com>
@kylecarbs
Copy link
Member

my linting was all wack so fixing this in main directly ty

@kylecarbs kylecarbs closed this Dec 3, 2025
@kylecarbs kylecarbs deleted the fix-vim-jk-keybindings branch December 3, 2025 17:38
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