Skip to content

fix: respect cursorlineopt in the diff and the list surfaces - #59

Merged
undont merged 1 commit into
mainfrom
fix/cursorlineopt
Sep 11, 2026
Merged

undont merged 1 commit into
mainfrom
fix/cursorlineopt

Conversation

@undont

@undont undont commented Sep 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • the diff's cursor-line overlay painted over the text whenever cursorline was on, even with cursorlineopt=number. it now paints only when cursorlineopt includes line, screenline or both; screenline still tints the whole line, since an extmark can't target one screen row
  • the OptionSet repaint also fires on cursorlineopt, so changing it mid-session updates the overlay straight away
  • the panel, history and pr checks lists set cursorlineopt=line locally, so a global number-only setting no longer hides their selected row

Test plan

  • make check clean

@undont
undont merged commit b7ab902 into main Sep 11, 2026
11 checks passed
@undont
undont deleted the fix/cursorlineopt branch September 11, 2026 09:10
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.

1 participant