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

ov changes cursor in vscode terminal #453

Open
Crissante opened this issue Oct 11, 2023 · 3 comments
Open

ov changes cursor in vscode terminal #453

Crissante opened this issue Oct 11, 2023 · 3 comments
Labels
bug Something isn't working question Further information is requested

Comments

@Crissante
Copy link

In vscode terminal with the setting

"terminal.integrated.cursorStyle": "line"

when invoking ov on a file (make sure that it is long enough so that it stays in pager mode) then quitting, the cursor changes from the expected line type to block

@noborus
Copy link
Owner

noborus commented Oct 11, 2023

Thank you for the issue.
This issue gdamore/tcell#623 I think it's due to.
ov is not changing the cursor, so I would like to wait for a fix in tcell.

@Crissante
Copy link
Author

Is there a temporary solution that you can think of?

@noborus
Copy link
Owner

noborus commented Oct 12, 2023

I looked into it again, and it seems that the cursor changes in vscode and doesn't return is a vscode issue.
microsoft/vscode#117741

For now, even if you set it to the default, it will not return, so you will need to set the line style again.
It would be a good idea to create a wrapper script so that the following is executed after ov finishes.

Power-Shell

Write-host "$([char]0x1B)[6 q"

Bash,etc...

 echo -e "\033[6 q"

@noborus noborus added bug Something isn't working question Further information is requested labels Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants