fix: restore terminal cursor style on exit #1299
Open
+12
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#1298
Fixes an issue where the cursor style was not properly restored when
exiting crush, particularly affecting terminals like urxvt with tmux.
Previously, when crush exited, the cursor would remain as a pink
vertical bar instead of returning to the user's configured cursor
style (typically a blinking block).
This fix adds a restoreCursor() function that:
escape sequences
The fix uses the charmbracelet/x/ansi package to send the proper
DECSCUSR escape sequence (CSI 0 SP q) to reset the cursor style
and show the cursor on program exit.
This is my first contribution to this repository. Please let me
know if I'm missing anything or if there are any additional changes
needed to align with project conventions.