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

Center the picker preview selection using visual lines #7837

Merged
merged 1 commit into from
Aug 8, 2023

Conversation

doy
Copy link
Contributor

@doy doy commented Aug 5, 2023

this way the preview always shows the selection even if lines were wrapped

fixes #7836

pascalkuthe
pascalkuthe previously approved these changes Aug 5, 2023
Copy link
Member

@pascalkuthe pascalkuthe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small not, otherwise thus liims good. This was a good catch and something I missed in #5420

helix-term/src/ui/picker.rs Outdated Show resolved Hide resolved
this way the preview always shows the selection even if lines were
wrapped
@doy
Copy link
Contributor Author

doy commented Aug 5, 2023

fixed

@pascalkuthe pascalkuthe added this to the next milestone Aug 6, 2023
@pascalkuthe pascalkuthe added C-bug Category: This is a bug E-easy Call for participation: Experience needed to fix: Easy / not much S-waiting-on-review Status: Awaiting review from a maintainer. A-helix-term Area: Helix term improvements labels Aug 7, 2023
@the-mikedavis the-mikedavis changed the title center the preview selection using visual lines Center the picker preview selection using visual lines Aug 8, 2023
@the-mikedavis the-mikedavis merged commit a7a145a into helix-editor:master Aug 8, 2023
@pascalkuthe pascalkuthe mentioned this pull request Sep 12, 2023
dgkf pushed a commit to dgkf/helix that referenced this pull request Jan 30, 2024
…7837)

this way the preview always shows the selection even if lines were
wrapped
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
…7837)

this way the preview always shows the selection even if lines were
wrapped
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
…7837)

this way the preview always shows the selection even if lines were
wrapped
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-bug Category: This is a bug E-easy Call for participation: Experience needed to fix: Easy / not much S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

global search preview doesn't correctly display the found line
3 participants