umputun / revdiff / 24009015554

90%

Build Type

push

github

Committed by web-flow

Commit Message

feat(ui): add toggleable line numbers in diff gutterLine numbers (#21)

* feat(ui): add lineNumbers toggle field and L key dispatch

* feat: compute dynamic line number column width per file

* feat: add lineNumGutter helper for formatted number columns

* feat: integrate line numbers into renderDiffLine

* feat: integrate line numbers into wrapped diff rendering

* feat: integrate line numbers into collapsed mode rendering

* feat: add # status icon and L help entry for line numbers

* test: add end-to-end integration test for line numbers

* fix: address code review findings

- Fix horizontal scroll cut width not accounting for line number gutter
  width in renderDiffLine, renderCollapsedAddLine, renderDeletePlaceholder
- Fix cursorViewportY delete-placeholder wrap width ignoring line numbers
- Update CLAUDE.md with # status icon, L key dispatch, line number data flow
- Update README.md and plugin usage.md with L keybinding

* fix: address codex review findings

* fix: address linter issues and misleading comment in lineNumGutter

Rename var old/new to oldCol/newCol to avoid shadowing Go builtin.
Use tagged switch on dl.ChangeType instead of untagged switch.
Fix comment: gutter uses lipgloss style, not raw ANSI.

* fix: gate toggleLineNumbers on diff pane focus

Align with toggleWrapMode/toggleCollapsedMode by checking
m.focus == paneDiff before toggling, so pressing L while
the tree pane is focused is a no-op.

---------

Co-authored-by: rashpile <koptilin@gmail.com>

Coverage Stats

115 of 149 new or added lines in 4 files covered. (77.18%)

1 existing line in 1 file now uncovered.

3300 of 3579 relevant lines covered (92.2%)

470.37 hits per line