cursor.hide() isn't scoped to the current screen reference about crossterm HOT 7 CLOSED

Thanks for creating the issue. This should be screen specific I need. However it is supposed to do that. I'll give it a look why this is not happening.

from crossterm.

    let screen = crossterm::Screen::default();
    match screen.enable_alternate_modes(false) {
        Ok(alternative) => {
            let ct = crossterm::Crossterm::from_screen(&alternative.screen);
            let cursor = ct.cursor();
            cursor.hide();
        }
        Err(e) => {}

I can't find why this would occur. I tested in on windows and got the same result as you. I still need to give it a test on a UNIX based machine and see if it is the same there. Also, I might chek if termion has the same problem.

from crossterm.

I think this issue is due to not restoring the previous console/terminal settings on shutdown. When applying syscalls to hide or show cursor, it needs to be reversed on exit -- checkout termbox-go's implementation.

from crossterm.

yes, that might be true. Did you check this for Unix systems? For WinApi it also seems like it operates terminal based and not screen based.

from crossterm.

I didn't check exactly on Unix systems, but looking at termbox-go: https://github.com/nsf/termbox-go/blob/master/api.go#L129

Notice how the code, upon closing, re-establishes some default state like showing the cursor back into stdout. Not sure if Unix-y systems has an equivalent to winapi's GetConsoleState, but it would be preferable to restore the preexisting state, rather than overwriting state (even if they are sane defaults)

from crossterm.

Ah, oke. It is just resetting everything back to default.

from crossterm.

Closing in favor of https://github.com/TimonPost/crossterm/issues/new

from crossterm.