hrvach / deskhop

Fast Desktop Switching Device
GNU General Public License v3.0
6.37k stars 185 forks source link

Configuration interface not working #136

Closed github-k8n closed 1 week ago

github-k8n commented 3 months ago

Updated both RPis using the "hold button on pi" method and tried with both self-compiled uf2 as well as 0.62 prebuilt image. Things seem to work, it still switches back and forth between the monitors, but it does not react at all to the configuration "Left Shift + Right Shift + C + O" , no reboot, no USB mass storage device, no blinking...

hrvach commented 3 months ago

Maybe the keyboard used is unable to send both shifts at the same time? If you have the self-compiled one, perhaps try changing the shortcut (in keyboard.c) to e.g. left control + right shift + c + o and rebuilding? Let me know if it still doesn't work.

SeanDennis101 commented 3 months ago

I Probably need to compile instead of using the prebuilt image because I am having the same issue as the other commenter. Will try a different keyboard first to make sure. The post comment from last week from github-k8n describes it exactly basically same exact issue.

github-k8n commented 3 months ago

Worked after I changed it to right Ctrl + right Shift + C + O

FYI the keyboards involved are

Neither of those worked. Not sure if detecting/sending both at the same time is something that only recent keyboards can do but maybe another combination might be better for the default setting...

hrvach commented 3 months ago

Ok, this is a good enough reason to change that key combo. I obviously can't choose a good one for the life of me, there is always something not working somewhere :)

Thank you for reporting this, fix should be in the next release.

jalmeroth commented 1 week ago

Does this issue persist or can we close this issue?

github-k8n commented 1 week ago

Sorry, yes the new default key combo works for me... thanks, can be closed.

jalmeroth commented 1 week ago

Cool, thanks for confirming! You can close it yourself or @hrvach has to do it 😉