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

Validate launcher - Window Walker reacts to winkey + mouse button #2097

Closed
savchenko opened this issue Apr 12, 2020 · 14 comments
Closed

Validate launcher - Window Walker reacts to winkey + mouse button #2097

savchenko opened this issue Apr 12, 2020 · 14 comments
Labels
Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something Product-PowerToys Run Improved app launch PT Run (Win+R) Window

Comments

@savchenko
Copy link

Environment

Windows build number: 10.0.18363.720
PowerToys version: v0.16.1
PowerToy module for which you are reporting the bug (if applicable): Window Walker

Steps to reproduce

  1. Launch PowerToys with WindowWalker module enabled
  2. Launch https://github.com/stefansundin/altdrag (Alt+Drag or Windows+Drag to move windows #269) that is using Winkey as the "control key".
  3. Observe WindowWalker appearing when winkey+mouse button are pressed.

Expected behavior

  1. WindowWalker should not pop-up unless ctrl+win are pressed simultaneously.
  2. WindowWalker should not react to mouse button events

Actual behavior

  1. WindowWalked pops-up when winkey+mouse button are pressed.

Screenshots

N/A, happy to supply a screen recording if needs be.

@crutkas
Copy link
Member

crutkas commented Apr 14, 2020

i think we should validate this with launcher since WW will migrate over

@crutkas crutkas added this to the Build 2020 milestone Apr 14, 2020
@crutkas crutkas changed the title Window Walker reacts to winkey + mouse button Validate launcher - Window Walker reacts to winkey + mouse button Apr 14, 2020
@savchenko
Copy link
Author

Thanks, I will test with https://github.com/microsoft/PowerToys/milestone/9 once it is released.

@crutkas
Copy link
Member

crutkas commented Jul 14, 2020

@asvc, does this work?

@crutkas crutkas removed this from the Build 2020 milestone Jul 14, 2020
@crutkas crutkas added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jul 14, 2020
@savchenko
Copy link
Author

savchenko commented Jul 14, 2020

@crutkas,
Downloaded v0.19.1. Would it be safe to assume that "WindowWalker" was renamed to the "PowerToys Run"? In that case it doesn't work with AltDrag, no new interfaces appear on the hotkey press. Changing the shortcut (I have tried alt+shift+space) makes no difference.

AltDrag is built from https://github.com/asvc/altdrag

@ghost ghost added Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jul 14, 2020
@crutkas
Copy link
Member

crutkas commented Jul 21, 2020

looking at AltDrag, it is GPLv3 so we can't look at source code which could make debugging hard. We're just using standard hooks here as are the other included utilities.

so even with 0.19, you hit winkey+mouse and it triggers?

@crutkas crutkas added Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something and removed Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Jul 21, 2020
@crutkas
Copy link
Member

crutkas commented Jul 21, 2020

@betsegaw window walker used a standard .net keyhook, right?

@betsegaw
Copy link
Contributor

betsegaw commented Jul 21, 2020

@crutkas Window Walker used the "RegisterHotKey" win32 API.

It adds a hook to look at windows messages and checks for a specific message.

@savchenko
Copy link
Author

looking at AltDrag, it is GPLv3 so we can't look at source code

@stefansundin , would you consider releasing under MIT, etc?

so even with 0.19, you hit winkey+mouse and it triggers?

@crutkas , using v0.19.1: "PowerToys Run" search bar doesn't appear after pressing an assigned hotkey while AltDrag is running.

@ghost ghost added Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams Needs-Team-Response An issue author responded so the team needs to follow up and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jul 22, 2020
@crutkas
Copy link
Member

crutkas commented Jul 27, 2020

@asvc, We're using standard keyhooks inside windows. They also do reg hacks it seems like and that could interfere based on the installer.

I did just install and try with an internal testing build for 0.20 and everything seems to work correctly.

@crutkas crutkas added Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something and removed Needs-Team-Response An issue author responded so the team needs to follow up Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Jul 27, 2020
@savchenko
Copy link
Author

@crutkas , I don't see v0.20 at https://github.com/microsoft/PowerToys/releases, is it from AppVeyor / etc?..

@ghost ghost added Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams Needs-Team-Response An issue author responded so the team needs to follow up and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jul 28, 2020
@crutkas
Copy link
Member

crutkas commented Jul 28, 2020

.20 will be released this month. We are testing internally. (Or you can clone master and build it yourself :))

@crutkas
Copy link
Member

crutkas commented Aug 4, 2020

@asvc we just released it on friday.

@crutkas crutkas added Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something and removed Needs-Team-Response An issue author responded so the team needs to follow up Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Aug 4, 2020
@ghost ghost added the Status-No recent activity no activity in the past 5 days when follow up's are needed label Aug 9, 2020
@ghost
Copy link

ghost commented Aug 9, 2020

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 5 days. It will be closed if no further activity occurs within 5 days of this comment.

@ghost ghost closed this as completed Aug 15, 2020
@savchenko
Copy link
Author

Just wanted to confirm that v0.20.1 works fine and AltDrag isn't affected by FancyZones. Thanks @crutkas !

@ghost ghost removed the Status-No recent activity no activity in the past 5 days when follow up's are needed label Aug 20, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something Product-PowerToys Run Improved app launch PT Run (Win+R) Window
Projects
None yet
Development

No branches or pull requests

4 participants