-
-
Notifications
You must be signed in to change notification settings - Fork 355
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
UI stays open after a Space transition, even when shortcut was released #588
Comments
6.4.0 negative. Still there |
Can confirm it doesn't happen for me either with apparition delay at 0. I know you mentioned (in the other thread) you only use a single space, but I've definitely only noticed this when switching apps triggers a space change (or from a fullscreen app to a non-fullscreen, or vice versa). |
The new release 6.7.0 should fix this issue. Closing this ticket. Let me know if the issue is not gone and I'll re-open it 👍 |
Sorry to say the issue is still there 😬 |
After investigating, I think what's happening is that during space transitions, macOS doesn't send some types of events to AltTab. Events are really complex in the current implementation, due to AltTab avoiding SecureInput. I noticed that during the transition:
As a workaround, I forced I wish I knew why macOS doesn't send us these 2 specific kinds of events during Space transitions. Here is a preview build. Could you please test it and let me know if that fixes the issue for you? 👍 |
It definitely reduces the problem, although it does still stay onscreen occasionally - maybe 1/10 times. Sometimes it simply stays onscreen as before, sometimes it manifests as #633. |
The problem went completely away for me with 6.7.1 |
## [6.7.2](v6.7.1...v6.7.2) (2020-10-06) ### Bug Fixes * crash in rare unknown scenario scenario ([08581f5](08581f5)) * crash on blacklisted app with main shortcut cleared ([c3f0686](c3f0686)) * ignore more non-user-facing apps (xpc processes) ([8417564](8417564)) * key repeat rate was too fast on high fps monitors (closes [#633](#633)) ([b408f14](b408f14)) * keynote was not showing while in slideshow mode (closes [#636](#636)) ([ec7b69f](ec7b69f)) * space transition sometimes absorbed the shortcut (closes [#588](#588)) ([5e6a0c2](5e6a0c2))
Just tried 6.7.2 and unfortunately still noticing the behaviour I mentioned in my last post - I think @MrBaseMax originally mentioned he'd only experienced the issue in same-space switches. I'm rolling back to 6.6.0 for now as I'm finding the key repeat aspect frustrating. For the most part I've trained myself to switch less quickly (but still quicker than I would be able with the default MacOS switcher), so generally I don't encounter the original issue as often any more. On the occasions when it does happen then dismissing the UI isn't that much extra hassle: I usually just hit the modifier again to switch to my expected window or escape to cancel the switch. However when combined with the stuck repeating trigger mentioned in #633, it disrupts workflow pretty badly - I end up switching to the final window, regardless of whether I hit the modifier or escape. Actually I've just noticed that in both 6.7.1 and 6.7.2 even without the key repeat being stuck and the UI just remaining on screen as per the original issue, hitting escape no longer cancels the switch - it always switches to the window highlighted in the list. |
I face no troubles (mentioned in this bug) when switching windows of different spaces. |
I dug deeper, and it's more of a mess than I thought. During the Space transition animation, macOS will not report some keyboard events. Here is a some scenario I'm able to reproduce fairly consistently:
You can see here that macOS is not sending AltTab the I realize now that my previous workaround couldn't work. I realize also that there is no way to compensate this missing event. It can impact the state of so many things, it's pretty crazy. The whole system relies on having every event arrive in order, as we maintain state of the keyboard to decide which shortcut was pressed/released and which action to take. I'm pretty blocked on how to deal with this missing event from the OS during Space animation. I'm also thinking that it could be the root cause of #633 as well. Note that I tried both these APIs: |
I may have found a workaround I think. I'm excited as I think I've fixed multiple issues. AltTab works great for me in these scenarios:
It all works for me! Can you please try out this local build and tell me if it works for you? |
I like. No issues yet |
Wow, that's fast work, I was just reading your previous response and thinking of all the events to be missed, that's possibly the most frustrating and difficult to compensate for. Tested the above scenarios and can confirm 1, 2, 5 + 6 seem much more robust / responsive, and the UI has correctly hidden every time. Can't comment on 3 as I've only got the single monitor. For 4, I'm never even sure if SecureInput is applicable or activated tbh - I tried clicking into a password field beforehand to see if it made any difference, but it didn't appear to do so: still hides consistently. A couple of other observations:
|
Fail how? It stays open again, or it gets absorbed and you don't see the other app being focused?
I think I've witnessed this a few times also when spamming keys like crazy. I can't make it happen slowly for instance. If you find reproduction steps for this let me know. Today I can't really reproduce it on purpose. Ok so overall it seems like a great improvement, no? I think I'll release it |
Sorry, the UI now always hides correctly but in these very rare circumstances the shortcut seems to be absorbed without focusing the next app. It really is very rare, only happening occasionally amongst multiple unrealistically fast key spams.
Weirdly, I'm having the exact same experience as you in that it seems even less prevalent today than when I tested yesterday. I think when it did happen on slow presses it was shortly after I'd witnessed it through spamming, so it seems unlikely to come up through normal usage. I'll keep an eye out if I do notice a bona fide pattern.
Definitely, hugely. Thanks for all the effort on this and the app overall - apologies for not mentioning that in my overzealous testing! |
From this other thread:
Bringing in @MrBaseMax's comment from #587:
I can consistently reproduce this one in these circumstances:
The text was updated successfully, but these errors were encountered: