Skip to content

fix: parse args before mousetrack#181

Merged
thomas-mauran merged 1 commit intomainfrom
fix/crossterm-mousetracking-bug
Dec 19, 2025
Merged

fix: parse args before mousetrack#181
thomas-mauran merged 1 commit intomainfrom
fix/crossterm-mousetracking-bug

Conversation

@thomas-mauran
Copy link
Owner

parse args before enabling mouse tracking

Description

right now when we do --version or any miss spelled commands we dont quit the mouse capture making the terminal unusable

Fixes #176

How Has This Been Tested?

manually

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Signed-off-by: Thomas Mauran <thomasmauran@yahoo.com>
@thomas-mauran thomas-mauran self-assigned this Dec 18, 2025
@thomas-mauran thomas-mauran added the bug Something isn't working label Dec 18, 2025
@thomas-mauran
Copy link
Owner Author

Hello @craftmaster1231 this should patch the issue if you want to try it out on your device too.
On a basic arch linux vm of mine and mac os this seems to patch the issue.

@craftmaster1231
Copy link

@thomas-mauran
Tested this branch build.
I confirm fix of #176 on my machine.

@thomas-mauran
Copy link
Owner Author

Beautiful thanks again @craftmaster1231 merging that on main

@thomas-mauran thomas-mauran merged commit cb34554 into main Dec 19, 2025
23 of 32 checks passed
@thomas-mauran thomas-mauran deleted the fix/crossterm-mousetracking-bug branch December 19, 2025 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mouse tracking left enabled after exit

2 participants