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

clear search when switch to insert mode #1135

Merged

Conversation

elmarsan
Copy link
Contributor

Description

There is a weird behavior when entering insert vi mode if you were previously doing searches.
When you search for a term an then switch to insert vi mode, the search term is not deleted. So the search term continues being evaluated if you still using the terminal.

Demo of current behavior

vi-search-mode.webm
Clear search term when switch to insert vi mode

Motivation and Context

Do not execute search term while using insert vi mode

How Has This Been Tested?

  • Manually
  • Unit tests

Checklist:

Go over all the following points, and put an x in all the boxes that apply.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

  • I have read the CONTRIBUTING document in my spoken language, and understand the terms
  • I have updated (or added) the documentation accordingly.
  • I have added tests to cover my changes.
  • I have gone through all the steps, and have thoroughly read the instructions

@github-actions github-actions bot added VT: Backend Virtual Terminal Backend (libterminal API) test Unit tests labels Jul 13, 2023
Copy link
Member

@christianparpart christianparpart left a comment

Choose a reason for hiding this comment

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

I actually liked that behavior but I totally understand why it may be confusing.

Can you add a changeling entry for this into the metainfo.xml?

@Yaraslaut
Copy link
Member

I actually liked that behavior but I totally understand why it may be confusing.

Can you add a changeling entry for this into the metainfo.xml?

Maybe we want to make it configurable then? Then you both can have desired behavior, right ?

@christianparpart
Copy link
Member

But if it is just me. I do not depend on it

@elmarsan
Copy link
Contributor Author

I believe the best option is make it configurable. For new users like me is weird, but I can imagine long term users might miss the original behavior.

Shall I make it configurable?

Copy link
Member

@christianparpart christianparpart left a comment

Choose a reason for hiding this comment

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

Many thanks, @elmarsan.

@Yaraslaut Yaraslaut merged commit e60326e into contour-terminal:master Jul 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Unit tests VT: Backend Virtual Terminal Backend (libterminal API)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants