Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: Add setting Cursor Surrounding Lines (#1474) #1475

Merged
merged 1 commit into from
Oct 13, 2022

Conversation

maxwell8888
Copy link
Contributor

This is the minimum number of visible lines above and below the cursor. The default is 1 as per the editors current behavior.

  • [ x] Added an entry to CHANGELOG.md if this change could be valuable to users

@codecov-commenter
Copy link

Codecov Report

Merging #1475 (813c753) into master (d5560f8) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master   #1475      +/-   ##
=========================================
- Coverage    6.04%   6.04%   -0.01%     
=========================================
  Files         124     124              
  Lines       50863   50867       +4     
=========================================
  Hits         3077    3077              
- Misses      47786   47790       +4     
Impacted Files Coverage Δ
lapce-data/src/config.rs 0.00% <ø> (ø)
lapce-ui/src/editor/view.rs 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dzhou121
Copy link
Collaborator

Sorry can you fix conflict for CHANGELOG.md

This is the minimum number of visible lines above and below the cursor.
The default is 1 as per the editors current behavior.
@maxwell8888 maxwell8888 force-pushed the feat/cursor-surrounding-lines branch from 813c753 to d90449b Compare October 12, 2022 21:21
@dzhou121 dzhou121 merged commit 611b189 into lapce:master Oct 13, 2022
dzhou121 pushed a commit that referenced this pull request Oct 13, 2022
This is the minimum number of visible lines above and below the cursor.
The default is 1 as per the editors current behavior.
@panekj panekj added this to the v0.2.2 milestone Jun 24, 2023
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.

4 participants