pmb6tz / windows-desktop-switcher

An AutoHotKey script for Windows that lets a user change virtual desktops by pressing CapsLock + <num>.
MIT License
1.23k stars 230 forks source link

Unable to move to other windows. #37

Closed NicTanghe closed 5 years ago

NicTanghe commented 5 years ago

I can not use caps + azerty to move active windows to another destop. It simply behaves the same as pressing caps + 1234.

Elijas commented 5 years ago

I did not understand the issue. I assume, you are using AZERTY keyboard instead of QWERTY and pressing Caps + A does not move the window to the first desktop?

If yes, could you try changing q in CapsLock & q::MoveCurrentWindowToDesktop(1) (link) to a?

NicTanghe commented 5 years ago

This is how that part of the user config looks.

CapsLock & a::MoveCurrentWindowToDesktop(1)
CapsLock & z::MoveCurrentWindowToDesktop(2)
CapsLock & e::MoveCurrentWindowToDesktop(3)
CapsLock & r::MoveCurrentWindowToDesktop(4)
CapsLock & t::MoveCurrentWindowToDesktop(5)
CapsLock & y::MoveCurrentWindowToDesktop(6)
CapsLock & u::MoveCurrentWindowToDesktop(7)
CapsLock & i::MoveCurrentWindowToDesktop(8)
CapsLock & o::MoveCurrentWindowToDesktop(9)

(ah troubleshooting always making people go. Do You Think i'm stupid XD)

dylanpjx commented 5 years ago

I have the same issue, I haven't remapped anything, so it's still capsLock+ qwerty and it works like capslock + number instead of it's intended function.

Elijas commented 5 years ago

I currently can't test this, but I wonder what the result of deleting the line with "switchDesktopByNumber" results in? Also what windows version (build number) do you have?

MoveCurrentWindowToDesktop(desktopNumber) { WinGet, activeHwnd, ID, A DllCall(MoveWindowToDesktopNumberProc, UInt, activeHwnd, UInt, desktopNumber - 1) switchDesktopByNumber(desktopNumber) }

On Sat, Jun 8, 2019, 08:05 Dylan notifications@github.com wrote:

I have the same issue, I haven't remapped anything, so it's still capsLock+ qwerty and it works like capslock + number instead of it's intended function.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pmb6tz/windows-desktop-switcher/issues/37?email_source=notifications&email_token=AA7FJFKGAR7HYU6MWF5QDMDPZM4YRA5CNFSM4HOB74U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXHNVGI#issuecomment-500095641, or mute the thread https://github.com/notifications/unsubscribe-auth/AA7FJFP2TMNZOM33V4IBCCTPZM4YRANCNFSM4HOB74UQ .

dylanpjx commented 5 years ago

I commented out the switchDesktopByNumber lines, and the moveCurrentWindowToDesktop still doesn't work. Do I need to install the .dll file for it to work? Because that wasn't included in the installation readme. I'm currently on Windows V1809, build 17763.529

Elijas commented 5 years ago

I commented out the switchDesktopByNumber lines, and the moveCurrentWindowToDesktop still doesn't work. Do I need to install the .dll file for it to work? Because that wasn't included in the installation readme. I'm currently on Windows V1809, build 17763.529

moveCurrentWindowToDesktop is completed by the DLL, so you're correct to identify it as the most likely issue.

No, install is not needed because dll is loaded directly as a file.

The script was tested with V1803, so it is likely the case that new Windows version broke the DLL.

justinmklam commented 5 years ago

Also facing this issue of being unable to move active windows to other desktops. Currently on Windows 1809 Build 17763.557.

nchowning commented 5 years ago

I'm having the same issue as of OS build 18362.207

nchowning commented 5 years ago

I pulled down the latest source for VirtualDesktopAccessor - https://github.com/Ciantic/VirtualDesktopAccessor - and built it. That resolved the issue for me

Elijas commented 5 years ago

@nchowning I'm glad to hear! Could you submit the dll via pull request?

dylanpjx commented 5 years ago

I think the github repo should include the building of the .dll as part of the readme, or at least link it to the virtualDesktopAccessor above. That would help in clarifying this issue since quite a few people seem to be confused about it. Cheers!

pmb6tz commented 5 years ago

Just pushed the requested change; fixed the issue for me. Give a try and let me know if it works for you as well. Thanks @nchowning !