Skip to content

Allow file navigation while in focused-file mode - #42

Open
tallshort wants to merge 1 commit into
cmpadden:mainfrom
tallshort:fix/focused-file-navigation
Open

tallshort wants to merge 1 commit into
cmpadden:mainfrom
tallshort:fix/focused-file-navigation

Conversation

@tallshort

Copy link
Copy Markdown

Closes #39.

Summary

Allow [ and ] to navigate between files while focused-file mode is active.

Previously, focusing a file with f limited file navigation to the currently visible file. This change uses the full
file index for navigation and updates the focused file when moving to another file, while retaining the single-file view.

Changes

  • Allow [ and ] to switch files without leaving focused-file mode
  • Preserve the existing first/last-file boundary behavior
  • Update the help text and README
  • Add a regression test covering navigation in both directions

Validation

  • npm test
  • npm run typecheck
  • npx prettier --check README.md src/review/component.ts test/review/component.test.ts

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.

Allow file navigation while in focused-file mode

1 participant