Add "monitor": "any"|"toCurrent"|"toMouse" setting to globalSummon

[zadjii-msft]

⚠️ this pr targets #9977

Summary of the Pull Request

This adds support for part of the monitor property for globalSummon. It also goes a little off-spec:

"monitor": "any"|"toCurrent"|"toMouse"

• monitor: This controls the monitor that the window will be summoned from/to
  • "any": Summon the MRU window, regardless of which monitor it's currently on.
  • "toCurrent"/omitted: (default): Summon the MRU window TO the monitor with the current foreground window.
  • [NEW] "toMouse": Summon the MRU window TO the monitor where the mouse cursor is.

When I was playing with this, It felt like toMouse was always what I wanted, not toCurrent. We can always just comment that out if we think that's contentious - I'm aware I didn't originally spec that.

References

• Original thread: Feature request: hot key drop down ala quake/guake/tilda #653
• Spec: Spec for Quake Mode #9274
• megathread: [MEGATHREAD] Quake Mode & Global Summon #8888

PR Checklist

• Closes https://github.com/microsoft/terminal/projects/5#card-60325291
• I work here
• Tests added/passed
• Requires documentation to be updated 😢

Detailed Description of the Pull Request / Additional comments

I made toMouse the default because it felt better. fite-me.jpg

Validation Steps Performed

my ever evolving blob:

    { "keys": "ctrl+`", "command": { "action": "quakeMode" } },
    { "keys": "ctrl+1", "command": { "action": "globalSummon" } },
    // { "keys": "ctrl+2", "command": { "action": "globalSummon", "desktop": "toCurrent" } },
    // { "keys": "ctrl+2", "command": { "action": "globalSummon", "toggleVisibility": false } },
    // { "keys": "ctrl+2", "command": { "action": "globalSummon", "dropdownDuration": 2000 } },
    { "keys": "ctrl+2", "command": { "action": "globalSummon", "monitor": "any" } },
    // { "keys": "ctrl+3", "command": { "action": "globalSummon", "desktop": "onCurrent" } },
    { "keys": "ctrl+3", "command": { "action": "globalSummon", "monitor": "toMouse" } },
    // { "keys": "ctrl+4", "command": { "action": "globalSummon", "desktop": "any" } },
    { "keys": "ctrl+4", "command": { "action": "globalSummon", "monitor": "toMouse", "dropdownDuration": 500 } },
    { "keys": "ctrl+5", "command": { "action": "globalSummon", "dropdownDuration": 500 } },

[miniksa]

Man I make one comment in bug bash and here it is! Nice!

[DHowett]

i know i asked you not to but also I don't care that much 😄

[DHowett]

make sure you don't automerge the base! 😄

[ghost]

Hello @zadjii-msft!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

[ghost]

🎉Windows Terminal Preview v1.9.1445.0 has been released which incorporates this pull request.:tada:

Handy links:

• Release Notes
• Store Download