fixes jump-window

[agzam]

Bug discovered while testing #149. Upstream. Looks like it's broken in Hammerspoon. Removed usage of the experimental module.

#149 (comment)

[jaidetree]

Gave this fix a shot but doesn't seem to be working. Seems like without the filter, it's unable to find the intended target.

[Grazfather]

What version of Hammerspoon did you discover this on? I noticed there was an update just yesterday.

[agzam]

I'm using Hammerspoon Version 0.9.91 (6078)
I have not upgraded OS to Monterey yet, supposedly they made some fixes for it.
I guess it broke very recently. Then let's not merge this thing just yet.
Like Sun Tzu would've said: "If you wait by the river long enough, you may see fixed bugs floating by."

[HyunggyuJang]

I encountered the same issue, and was able to resolve with HyunggyuJang@dbb12cd

I’m not sure it is the canonical solution and works for all, so I’d like to know your opinions about it!

[agzam]

For me - both cases work. And whatever is in the master branch - doesn't. Even though I'm now using version 0.9.93 (6148) - it's still broken. At this point, I think we have to merge a fix - using either method. However, I've noticed another thing - after jumping to the target window, Spacehammer menu doesn't go away. That looks like a regression. I'll check what's going on there.

[Grazfather]

I can't repro on 6172. Do you still have this issue?

[agzam]

I can't repro on 6172. Do you still have this issue?

The latest available release is 0.9.93 (6148) - both on GitHub and via brew. How did you get 6172?

[Grazfather]

It was created to test a fix for an issue I reported Hammerspoon/hammerspoon#3081 https://github.com/Hammerspoon/hammerspoon/actions/runs/1726174041

[Grazfather]

Is this still needed?

[agzam]

Is this still needed?

Yes, jump to window appears broken in master. But this "fix" doesn't seem to be fixing it for good. It works, but not reliably.