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

WinKey for launcher throws a lot of false positives #4578

Closed
crutkas opened this issue Jun 29, 2020 · 16 comments
Closed

WinKey for launcher throws a lot of false positives #4578

crutkas opened this issue Jun 29, 2020 · 16 comments
Assignees
Labels
Hot Fix Items we will product an out-of-band release for Issue-Bug Something isn't working Priority-0 Bugs that we consider release-blocking/recall-class (P0) 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

@crutkas
Copy link
Member

crutkas commented Jun 29, 2020

Environment

Windows build number: [run "ver" at a command prompt]
PowerToys version: 0.19
PowerToy module for which you are reporting the bug (if applicable): PT Run

Steps to reproduce

Remap key from alt-space to win-space
execute shortcut multiple times

Expected behavior

don't have start menu false positive executions

Actual behavior

Lots of start menu executions

@crutkas crutkas added Issue-Bug Something isn't working Product-PowerToys Run Improved app launch PT Run (Win+R) Window Priority-0 Bugs that we consider release-blocking/recall-class (P0) labels Jun 29, 2020
@crutkas crutkas added this to the InVEST-2007 milestone Jun 29, 2020
@ghost ghost added the Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams label Jun 29, 2020
@crutkas crutkas removed the Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams label Jun 29, 2020
@arjunbalgovind
Copy link
Contributor

I'm able to hit this only if space gets pressed before the win key. Or when I press them both at the same time but space was by chance pressed earlier

@crutkas
Copy link
Member Author

crutkas commented Jun 29, 2020

i find if you do it pretty quick, it triggers. Same speed i would hit Win+R or Win+E

@ThoAppelsin
Copy link

I can confirm this.

I had been using Win+Space to switch between keyboard languages. I have never once seen the Start Menu triggering with that. I have just tried PT Run with Win+Space, and this happened on my first try, and consistently later on.

@crutkas
Copy link
Member Author

crutkas commented Jul 1, 2020

from #4666 looks like we get some false positives for even alt-space

@yuyoyuppe yuyoyuppe self-assigned this Jul 2, 2020
@yuyoyuppe yuyoyuppe added the Status-In progress This issue or work-item is under development label Jul 2, 2020
@enricogior
Copy link
Contributor

We are going to test a possible fix, replacing the low level hook with the Win32 API RegisterHotkey (that we use for the FZ editor and doesn't have those type of problems).

@arjunbalgovind
Copy link
Contributor

@enricogior will that work for overriding windows shortcuts like Win+R/Win+Space? Launcher used to use the NHotkey API earlier which internally used the Win32 RegisterHotkey API.

@enricogior
Copy link
Contributor

@arjunbalgovind
after talking with @crutkas as well, since we need to support those shortcuts, we will keep using the low level hook, but with a different approach (similarly to what FZ does for the Win + Arrow hotkey).

@arjunbalgovind
Copy link
Contributor

Agreed. I think the event-queue based approach added in #3825 could have threading related issues so using the FZ approach would be better.

@crutkas
Copy link
Member Author

crutkas commented Jul 7, 2020

Fix committed in 0.19.1. https://github.com/microsoft/PowerToys/releases/tag/v0.19.1

@crutkas crutkas closed this as completed Jul 7, 2020
@Bandit
Copy link

Bandit commented Jul 7, 2020

FWIW I'm still getting this issue - seems to be if I press space within ~150ms of holding down the Windows key, it opens the start menu. If I hold the Windows key, wait a split second, then try, it works reliably. Using a Satechi Mac keyboard (so my Windows key is a CMD key if that makes any difference).

@xt0rted
Copy link

xt0rted commented Jul 7, 2020

Still seeing this as well. I've been using AutoHotkey to map Win + Space to F13 and then I use that to trigger Wox. Until PowerToys Run can match this experience, as well as offer a setting to clear the last search term, I'll be sticking with this setup.

@josh-the-tech
Copy link

josh-the-tech commented Jul 8, 2020

Sadly still getting it too and just updated to v0.19.1. See my awful gif example!
2020-07-08_18-10-00

@arjunbalgovind
Copy link
Contributor

@Bandit I'm able to get Start menu to pop up like this if I press Win+Space, but I release the Win key before I release Space. I think that might be what's happening while hitting the shortcut quickly.

@carlocardella
Copy link
Member

carlocardella commented Jul 8, 2020

So far I am unable to reproduce with the default ALT+Space; I could repro at almost every keystroke with the previous release, it was impossible to type anything

@crutkas crutkas removed the Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release. label Jul 8, 2020
@crutkas crutkas reopened this Jul 8, 2020
@crutkas
Copy link
Member Author

crutkas commented Jul 8, 2020

We are reopening and a PR for a fix is under review now

@arjunbalgovind arjunbalgovind added Status-In progress This issue or work-item is under development 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 Jul 8, 2020
@crutkas
Copy link
Member Author

crutkas commented Jul 15, 2020

Fixed in 0.19.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hot Fix Items we will product an out-of-band release for Issue-Bug Something isn't working Priority-0 Bugs that we consider release-blocking/recall-class (P0) 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
None yet
Development

No branches or pull requests

9 participants