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

Ctrl+Mouse scroll changes the font size #3793

Open
alabuzhev opened this issue Dec 1, 2019 · 9 comments
Open

Ctrl+Mouse scroll changes the font size #3793

alabuzhev opened this issue Dec 1, 2019 · 9 comments
Labels
Area-Input Related to input processing (key presses, mouse, etc.) Area-Server Down in the muck of API call servicing, interprocess communication, eventing, etc. Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-2 A description (P2) Product-Conhost For issues in the Console codebase
Milestone

Comments

@alabuzhev
Copy link
Contributor

alabuzhev commented Dec 1, 2019

Environment

Windows build number: Microsoft Windows [Version 10.0.18363.476]
Windows Terminal version (if applicable): No

Any other software? No

Steps to reproduce

Run the attached app, press and hold Ctrl, start scrolling.

Expected behavior

The app receives MOUSE_EVENT_RECORDs and uses them to control the slider.

Actual behavior

The app receives MOUSE_EVENT_RECORDs and uses them to control the slider, but at the same time the host changes the font size, so the intended functionality is effectively unusable.

It does not happen in legacy mode and never happened prior to Windows 10.

There is no way to disable this feature in console settings - "Enable Ctrl key shortcuts" is already off.

UPD: everything above applies to Ctrl+Shift+Mouse Scroll as well. It changes the window transparency and there's no way to disable it.

@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Dec 1, 2019
@DHowett-MSFT DHowett-MSFT added Area-Input Related to input processing (key presses, mouse, etc.) Area-Server Down in the muck of API call servicing, interprocess communication, eventing, etc. Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Conhost For issues in the Console codebase labels Dec 2, 2019
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Dec 2, 2019
@DHowett-MSFT DHowett-MSFT removed the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Dec 2, 2019
@DHowett-MSFT DHowett-MSFT added this to the 21H1 milestone Dec 2, 2019
@bgshacklett
Copy link

Another unintended consequence is that non-precision touch-pads with momentum can result in the font size being changed unexpectedly.

Step 1: scroll with the touch-pad
Step 2: press the ctrl key shortly after you're finished scrolling
Step 3: marvel at your newly ginormous or insanely small text

This isn't a concern for many, but in environments with corporate-supplied laptops, which I would expect to be a relatively common deployment scenario for Windows Terminal, it's more likely to have a laptop without a precision touch-pad.

@DHowett
Copy link
Member

DHowett commented Mar 26, 2021

@bgshacklett unfortunately, that's true of so many applications. I wish it weren't 😦 but when I scroll in Firefox or Chrome or... well, pretty much any application and then hold down control I get my eyes melted off

@DHowett
Copy link
Member

DHowett commented Mar 26, 2021

WAIT, CHROME DOESN'T DO IT ANYMORE

IT'S FIXABLE???

@bgshacklett
Copy link

FWIW, Chrome still does it for me.

@zadjii-msft zadjii-msft added Help Wanted We encourage anyone to jump in on these. Priority-2 A description (P2) labels Jan 4, 2022
@zadjii-msft zadjii-msft modified the milestones: Windows vNext, Backlog Jan 4, 2022
@zadjii-msft zadjii-msft added Area-CookedRead The cmd.exe COOKED_READ handling and removed Area-CookedRead The cmd.exe COOKED_READ handling labels Feb 23, 2022
@esterkaufman
Copy link

It is happening with any app like webStorm, Chrome, Edge, and Rider.
Should it be fixed soon?

@artnikbrothers
Copy link

I happens in "Azure DevOps -> code review" as well. Very annoying issue

@marcelwgn
Copy link
Contributor

marcelwgn commented Apr 9, 2024

@alabuzhev I've tried the attached app and the font size does not change for me (Or i misunderstood the issue):

Slider in Terminal increasing and decreasing value

The transparency not increasing the value but rather changing opacity is still present though

Wrong product...

@alabuzhev
Copy link
Contributor Author

@marcelwgn this report is about conhost/openconsole, not WT.

@marcelwgn
Copy link
Contributor

marcelwgn commented Apr 9, 2024

So what do I need to watch out for when running the attached project?

Edit: Nevermind, figured it out...

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.) Area-Server Down in the muck of API call servicing, interprocess communication, eventing, etc. Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-2 A description (P2) Product-Conhost For issues in the Console codebase
Projects
None yet
Development

No branches or pull requests

8 participants