feat: add option to enable zoom with ctrl + scroll

[paulinek13]

Summary of the Pull Request

Introduces a setting to enable/disable font size changes in the terminal when holding Ctrl and scrolling, giving users more control over zoom actions.

References and Relevant Issues

#11710
#5708
#3793
#11906
#3990
#14627

Detailed Description of the Pull Request / Additional comments

This PR adds a new global setting scrollToZoom that allows users to enable font zooming with scrolling. When disabled, this setting prevents accidental font size changes that can occur when users scroll while holding the Ctrl key.

Note: after disabling this setting, users may still change font size using Ctrl+ and Ctrl- keyboard shortcuts. Other Ctrl+Scroll functionality (like transparency adjustments) remains unaffected.

Validation Steps Performed

• For testing, I built the project with Invoke-OpenConsoleBuild -m "-p:Configuration=Release,Platform=x64" and registered it using \repos\terminal\src\cascadia\CascadiaPackage\bin\x64\Release\AppxManifest.xml

• Verified the setting can be toggled in the Settings UI (Interaction tab)

• Confirmed that when disabled, holding Ctrl and scrolling no longer changes font size

• Validated that the setting persists across terminal restarts

PR Checklist

• Closes Request: Setting to disable zooming with ctrl+mousewheel #11710
• Closes Ctrl+Mouse scroll changes the font size #3793
• Closes Option to disable "Pinch to Zoom" #11906
• Closes Add a setting to disable mouse-wheel scrolling #3990
• Tests added/passed Summary: Total=278, Passed=278, Failed=0, Blocked=0, Not Run=0, Skipped=0
• Documentation updated
  • If checked, please file a pull request on our docs repo and link it here: Add documentation for experimental.scrollToZoom setting MicrosoftDocs/terminal#871
• Schema updated (if necessary)

---

Note: I used the existing FocusFollowMouse setting as a reference for implementing this.

[Lo0oG]

Just wondering if instead of being "Disable zooming with Ctrl+mouse wheel" the option was switched to the other way and was "Zoom with Ctrl+mouse wheel".
Avoids the double negative of Disable zooming with Ctrl+mouse wheel : Off.

[carlos-zamora]

The code itself is solid. Thanks for doing this! 😊

My only holdup is the wording used. I think it's worth avoiding the double negative as @Lo0oG mentioned.

Some other miscellaneous work here:

• please update the docs on our docs repo
• update the list of references and relevant issues to use the "Closes" keyword
  • this automatically closes the issues when the PR merges. Looks like you can close #11710, #3990, and #11906
  • 💡Pro Tip: "Closes #11710, #3990, #11906" should be smart enough to close all 3 issues, rather than repeated "Closes" over and over again EDIT: nope, you've gotta put "Closes" before each one

[DHowett]

💡Pro Tip: "Closes #11710, #3990, #11906" should be smart enough to close all 3 issues, rather than repeated "Closes" over and over again

@carlos-zamora it's actually not. It does not work, and the standard for our project is indeed to put each Closes on its own line.

[paulinek13]

Thanks for the reviews and comments! I've addressed them. I'll open a PR for the docs update shortly 😃

[carlos-zamora]

Looks good! Only thing is that we should set the default value to true throughout. That way, we don't accidentally disable it for everybody haha. But I think that's the last bit of feedback I have. Good work! 😊

[DHowett]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[DHowett]

@jsoref This is a weird case. This PR includes the spelling 0.0.25 changes from main, and it has run and passed, but all the Report phases are being "Skipped" (?) and we aren't getting "security scanning" results. I'm sure I'm just missing something, but... wat

[jsoref]

https://github.com/microsoft/terminal/actions/runs/16312069474/job/46126597737#step:2:12

use_sarif: false

I'd have to trace ...

https://github.com/microsoft/terminal/actions/runs/16312069474/workflow#L107

      use_sarif: ${{ (!github.event.pull_request || (github.event.pull_request.head.repo.full_name == github.repository)) && 1 }}

Sigh. Just change this to use_sarif: 1

I think I must have changed it for a different Microsoft repository.

[DHowett]

I'd have to trace ...

2025-07-16T20:09:15.2910498Z ##[debug]Evaluating: ((!github.event.pull_request || (github.event.pull_request.head.repo.full_name == github.repository)) && 1)
2025-07-16T20:09:15.2911586Z ##[debug]Evaluating And:
2025-07-16T20:09:15.2912149Z ##[debug]..Evaluating Or:
2025-07-16T20:09:15.2914436Z ##[debug]....Evaluating Not:
2025-07-16T20:09:15.2915743Z ##[debug]......Evaluating Index:
2025-07-16T20:09:15.2916303Z ##[debug]........Evaluating Index:
2025-07-16T20:09:15.2916843Z ##[debug]..........Evaluating github:
2025-07-16T20:09:15.2917429Z ##[debug]..........=> Object
2025-07-16T20:09:15.2917974Z ##[debug]..........Evaluating String:
2025-07-16T20:09:15.2918522Z ##[debug]..........=> 'event'
2025-07-16T20:09:15.2919046Z ##[debug]........=> Object
2025-07-16T20:09:15.2919538Z ##[debug]........Evaluating String:
2025-07-16T20:09:15.2920078Z ##[debug]........=> 'pull_request'
2025-07-16T20:09:15.2920697Z ##[debug]......=> Object
2025-07-16T20:09:15.2921178Z ##[debug]....=> false
2025-07-16T20:09:15.2921923Z ##[debug]....Evaluating Equal:
2025-07-16T20:09:15.2923183Z ##[debug]......Evaluating Index:
2025-07-16T20:09:15.2923740Z ##[debug]........Evaluating Index:
2025-07-16T20:09:15.2924479Z ##[debug]..........Evaluating Index:
2025-07-16T20:09:15.2925021Z ##[debug]............Evaluating Index:
2025-07-16T20:09:15.2925573Z ##[debug]..............Evaluating Index:
2025-07-16T20:09:15.2926209Z ##[debug]................Evaluating github:
2025-07-16T20:09:15.2926804Z ##[debug]................=> Object
2025-07-16T20:09:15.2927360Z ##[debug]................Evaluating String:
2025-07-16T20:09:15.2928193Z ##[debug]................=> 'event'
2025-07-16T20:09:15.2928771Z ##[debug]..............=> Object
2025-07-16T20:09:15.2929306Z ##[debug]..............Evaluating String:
2025-07-16T20:09:15.2929876Z ##[debug]..............=> 'pull_request'
2025-07-16T20:09:15.2930437Z ##[debug]............=> Object
2025-07-16T20:09:15.2930990Z ##[debug]............Evaluating String:
2025-07-16T20:09:15.2931725Z ##[debug]............=> 'head'
2025-07-16T20:09:15.2932296Z ##[debug]..........=> Object
2025-07-16T20:09:15.2932810Z ##[debug]..........Evaluating String:
2025-07-16T20:09:15.2933338Z ##[debug]..........=> 'repo'
2025-07-16T20:09:15.2933975Z ##[debug]........=> Object
2025-07-16T20:09:15.2934482Z ##[debug]........Evaluating String:
2025-07-16T20:09:15.2935012Z ##[debug]........=> 'full_name'
2025-07-16T20:09:15.2935615Z ##[debug]......=> 'paulinek13/terminal'
2025-07-16T20:09:15.2936198Z ##[debug]......Evaluating Index:
2025-07-16T20:09:15.2936720Z ##[debug]........Evaluating github:
2025-07-16T20:09:15.2937273Z ##[debug]........=> Object
2025-07-16T20:09:15.2937761Z ##[debug]........Evaluating String:
2025-07-16T20:09:15.2938299Z ##[debug]........=> 'repository'
2025-07-16T20:09:15.2938868Z ##[debug]......=> 'microsoft/terminal'
2025-07-16T20:09:15.2939859Z ##[debug]....=> false
2025-07-16T20:09:15.2940529Z ##[debug]..=> false
2025-07-16T20:09:15.2941140Z ##[debug]=> false
2025-07-16T20:09:15.2945400Z ##[debug]Expanded: ((!Object || ('paulinek13/terminal' == 'microsoft/terminal')) && 1)

[jsoref]

Yep, change that line to just use_sarif: 1 -- get rid of the fancy logic (the fancy logic can't work with rulesets).

I've updated https://docs.check-spelling.dev/Breaking-Change:-Code-Scanning-action-requires-a-Code-Scanning-Ruleset.html#migration

[DHowett]

Thanks @jsoref - just merged that in 🙂
check-spelling is still the GOAT

[DHowett]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[DHowett]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[DHowett]

thank you so much - folks are gonna love it