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

Mouse events are not sent when the mouse is outside the top of the WT #17744

Closed
BDisp opened this issue Aug 19, 2024 · 2 comments · Fixed by #17779
Closed

Mouse events are not sent when the mouse is outside the top of the WT #17744

BDisp opened this issue Aug 19, 2024 · 2 comments · Fixed by #17779
Labels
Area-Input Related to input processing (key presses, mouse, etc.) In-PR This issue has a related PR Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Tag-Fix Doesn't match tag requirements Product-Terminal The new Windows Terminal.

Comments

@BDisp
Copy link

BDisp commented Aug 19, 2024

Windows Terminal version

Versão: 1.20.11781.0

Windows build number

10.0.22631.0

Other Software

No response

Steps to reproduce

Please clone this GitHub repository MouseTests and run. Keep pressing the mouse inside the console and move around it, as the gif show.

Expected Behavior

The mouse events should be sent while the mouse button is pressed and moving inside or outside the console until it's released.

Actual Behavior

The mouse events are sent as expected except from outside of the console on top.

MouseTests

With the conhost the mouse events are always sent from anywhere while the mouse isn't released.

@BDisp BDisp added Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Aug 19, 2024
@carlos-zamora carlos-zamora added this to the Terminal v1.23 milestone Aug 21, 2024
@carlos-zamora carlos-zamora added Area-Input Related to input processing (key presses, mouse, etc.) Product-Terminal The new Windows Terminal. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Aug 21, 2024
@zadjii-msft
Copy link
Member

Yep, that's super weird alright!

Our theory is that we're getting into the mouse-select-drag-scroll state, and attempting to "scroll" the terminal, and suppressing the mouse event

@BDisp
Copy link
Author

BDisp commented Aug 21, 2024

Yep, I already noted that and that why I made my example writing always on the same line to avoiding scrolling. I noted that moving outside the console ws causing to scroll. This would be valid if I was pressed the scroll bar but that wasn't the case. Maybe acceptable only if the scroll bar is visible. Thanks.

@microsoft-github-policy-service microsoft-github-policy-service bot added the In-PR This issue has a related PR label Aug 22, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Tag-Fix Doesn't match tag requirements label Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Input Related to input processing (key presses, mouse, etc.) In-PR This issue has a related PR Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Tag-Fix Doesn't match tag requirements Product-Terminal The new Windows Terminal.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants