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

Use a boolean to indicate whether to yank during deletion of a selection #10132

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

voiceroy
Copy link
Contributor

@voiceroy voiceroy commented Apr 3, 2024

Closes #10050

Introduce an argument yank: bool in fn delete_selection_impl to indicate whether to yank the selection while deleting it, and update fn delete_selection, fn delete_selection_noyank, fn change_selection, fn change_selection_noyank accordingly.

@the-mikedavis the-mikedavis merged commit f773972 into helix-editor:master Apr 3, 2024
6 checks passed
Desdaemon pushed a commit to Desdaemon/helix that referenced this pull request Apr 3, 2024
@archseer
Copy link
Member

archseer commented Apr 4, 2024

I would have preferred an enum here :)

@voiceroy
Copy link
Contributor Author

voiceroy commented Apr 4, 2024

should I make another pr?

postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 4, 2024
postsolar pushed a commit to postsolar/helix that referenced this pull request Apr 20, 2024
mtoohey31 pushed a commit to mtoohey31/helix that referenced this pull request Jun 2, 2024
Vulpesx pushed a commit to Vulpesx/helix that referenced this pull request Jun 7, 2024
smortime pushed a commit to smortime/helix that referenced this pull request Jul 10, 2024
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.

keybind not working: delete_selection_noyank, search_prev
4 participants