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

[Run] "launch on monitor" setting for multimonitor user #1607

Closed
htcfreek opened this issue Mar 17, 2020 · 5 comments
Closed

[Run] "launch on monitor" setting for multimonitor user #1607

htcfreek opened this issue Mar 17, 2020 · 5 comments
Labels
Help Wanted We encourage anyone to jump in on these and submit a PR. Idea-Enhancement New feature or request on an existing product Product-PowerToys Run Improved app launch PT Run (Win+R) Window Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release.
Milestone

Comments

@htcfreek
Copy link
Collaborator

htcfreek commented Mar 17, 2020

Summary of the new feature/enhancement

A setting to define the start options of power launcher for multimonitor users would be nice.
Possible values are: default monitor, follow cursor [default value], display 1/2/3/...

Detailed description
In the specification of power launcher I found an entry who says: "For multiple monitors, the interface should follow where the mouse cursor is located". I interprated that the power launcher opens at the aktuall cursor position.

Not all Users want to open power launcher at the position of mouse cursor. For example I got used to the fact that the start menu opens every time on my main monitor.
Therefor It would be nice when the user can define the open behaviour of power launcher.

Proposed technical implementation details (optional)

I think there could be a dropdown menu with the following options in settings menu: at cursor position, default monitor, monitor 1/2/n
image

To help the user by chosing the monitor, you could show the number on the selected monitor while the cursor hover the dropdown entry.
image

Whe user opened the settings and the choosen monitor isn't available there should be shown a hint.
image

Other details:

  • The default behavior/value after installation could be "on cursor position".
  • For the options other than "cursor position" the powerlauncher should open centered on the screen.
  • If the choosen monitor (monitor 1, monitor 2, ...) is not avalable the power launcher could open as fallback at the cursor position.
@jyuwono jyuwono added Idea-Enhancement New feature or request on an existing product Product-PowerToys Run Improved app launch PT Run (Win+R) Window labels Mar 17, 2020
@jyuwono jyuwono added this to the Backlog milestone Mar 17, 2020
@crutkas
Copy link
Member

crutkas commented Mar 17, 2020

it wouldn't be at the cursor, it is "center of the monitor where the mouse currently is"

@addrum
Copy link
Contributor

addrum commented Feb 2, 2021

I'm willing to take this on. I've been having a play and have the window always on the primary monitor already; I'm just wondering if RememberLastLaunchLocation is used for anything? I can't find any set references so it seems to be a placeholder? If that's the case, I'm thinking I can repurpose it to hold the setting for where to display the window?

Oh also are there any UI tests? e.g. assert window is at position x,y - I can add to it if so or create new if not

@addrum
Copy link
Contributor

addrum commented Feb 2, 2021

I've just seen #4576 so obviously RememberLastLaunchLocation is for that. I'll create a separate setting for monitor.

@htcfreek htcfreek added the Status-In progress This issue or work-item is under development label Feb 4, 2021
@enricogior enricogior added Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release. and removed Status-In progress This issue or work-item is under development labels Mar 9, 2021
@htcfreek
Copy link
Collaborator Author

htcfreek commented Mar 23, 2021

For the implementation we have at the moment please see this comment: #10330 (comment).

@crutkas
Copy link
Member

crutkas commented Apr 1, 2021

This has been resolved in the 0.35 release. https://github.com/microsoft/PowerToys/releases/latest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help Wanted We encourage anyone to jump in on these and submit a PR. Idea-Enhancement New feature or request on an existing product Product-PowerToys Run Improved app launch PT Run (Win+R) Window Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release.
Projects
Status: ⚠️Needs Walkthrough
Development

No branches or pull requests

6 participants