-
-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
HOME shortcut not working correctly on scrcpy v1.15 #1643
Comments
v1.15 now forwards Ctrl (and Shift) to the device, so the shortcut modifier (MOD) had to be changed. By default, it's Alt or Super. See: |
I see, I read the changelog but I didn't realize the change. My bad... |
Oh, you're not the only one 😕 #1598 (comment) |
Did you have the same problem on v1.14? |
No problem on v1.14. |
That's very weird, I could not reproduce. Could you add this log, please: diff --git a/app/src/input_manager.c b/app/src/input_manager.c
index 02dd9cb5..cf557716 100644
--- a/app/src/input_manager.c
+++ b/app/src/input_manager.c
@@ -582,6 +582,7 @@ input_manager_process_mouse_button(struct input_manager *im,
const SDL_MouseButtonEvent *event) {
bool control = im->control;
+ LOGD("mouse button type=%d, which=%d, button=%d", event->type, event->which, event->button);
if (event->which == SDL_TOUCH_MOUSEID) {
// simulated from touch events, so it's a duplicate
return; And when middle click fails, please post the line it prints in the console. cc @AzureRaptor |
I'm sorry... Where should I add that? |
OK, here is a build:
(replace it in your v1.15 release) EDIT: run with |
This one didn't work from the start. I noticed it does work on the lockscreen but once the device is awake it stops detecting it (Alt-H isn't working either) I'm sorry I can't be more helpful with logs and stuff, I'm not very experienced |
Oh, I put a debug log, so it is not printed by default (should have added an info log for simplicity). Could you please run with Press Windows+r, type It should print logs when you press middle-click. Copy-paste them here. |
Nevermind, here is a build with an INFO log:
(replace it in your v1.15 release) |
When HOME stops working, what if you execute Press Windows+r, type |
the log says: INFO: mouse button type=1025, which=0, button=2 (when I press middle-click) adb shell input keyevent HOME worked |
Just in case I wasn't clear... the command worked at sending the home button, it didn't however fix the problem. Also, the latest adb packaged with scrcpy fails at connecting wirelessly, is doesn't seem to be related to scrcpy because it works if I replace it with the previous version of adb. |
Oh, I think I found a possible cause for the broken HOME shortcut. Here is a fix: 8fb0192
Does it fix the problem for you? @zequip @AzureRaptor |
Fixed!! |
Thank you. I will release a new version 1.15.1 soon then. |
I tested this morning on Windows, it works for me. |
no, thank YOU!! for the fix and for the amazing app. I let you decide if the thread should be left open or closed. |
I might be having some overlapping issue with the previous adb, then‽ It's weird because it works just fine while connected through USB... |
Disregard everything I said about ADB. I don't know exactly what I did but it's working now. |
Fixed in v1.15.1. |
I have read the FAQ.
I have searched in existing issues.
OS: windows 10
scrcpy version: 1.15
device model: Samsung galaxy j7
Android version: Android 8.1.0
All desktop shortcuts stopped working on the latest scrcpy release (V1.15), only left and right click are recognized. If I rollback to V1.14 everything works as intended.
EDIT: remaining issue: only the HOME shortcut is sometimes broken #1643 (comment)
The text was updated successfully, but these errors were encountered: