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

Always show cursor when DontBlinkCursor is true #12

Open
kiwiz opened this issue Jul 28, 2023 · 1 comment
Open

Always show cursor when DontBlinkCursor is true #12

kiwiz opened this issue Jul 28, 2023 · 1 comment

Comments

@kiwiz
Copy link

kiwiz commented Jul 28, 2023

Rather than updating the cursor after 250ms, I'd like for the option to immediately move the cursor on update (and not hide it). From a quick look at the code, it seems like I could just remove:

await Task.Delay(Settings.CursorBlinkInterval);
TerminalWindow.UnsetCursor(currentRow, currentCol);

in Program.cs. IMO, this would be nicer behaviour for DontBlinkCursor. Another option would be to add a flag for this new behaviour.

@i-am-shodan
Copy link
Owner

i-am-shodan commented Jul 28, 2023

That function doesn't hold state of where the cursor was in the past. So if you remove that code the line drawn for the cursor will never be removed from screen.

There need to be a bit of code to track this and always unset it when a new update comes along.

And that code need to be thread safe

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

No branches or pull requests

2 participants