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.

Sign up for GitHub

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

Jump to bottom

Fix shrink_selection with multiple cursors. #6093

Merged
merged 2 commits into from
Mar 9, 2023
Merged

Fix shrink_selection with multiple cursors. #6093

merged 2 commits into from
Mar 9, 2023

Commits on Feb 23, 2023

  1. Fix helix-editor#6092 

    Cause were some incorrect assumptions that missed an edge case in the
`Selection.contains()` calculation. Tests were added accordingly.
    @gibbz00
    gibbz00 committed Feb 23, 2023
    Configuration menu
    Copy the full SHA
    d93f9e7 View commit details
    Browse the repository at this point in the history

Commits on Feb 24, 2023

  1. Fix Selection.contains() edge-case handling. 

    Removing the len check short-circuit was the only thing needed as
pointed out by @dead10ck.
    @gibbz00
    gibbz00 committed Feb 24, 2023
    Configuration menu
    Copy the full SHA
    c95c142 View commit details
    Browse the repository at this point in the history