diff options
Diffstat (limited to 'gitlab/issues_text/target_missing/host_missing/accel_missing/558')
| -rw-r--r-- | gitlab/issues_text/target_missing/host_missing/accel_missing/558 | 57 |
1 files changed, 0 insertions, 57 deletions
diff --git a/gitlab/issues_text/target_missing/host_missing/accel_missing/558 b/gitlab/issues_text/target_missing/host_missing/accel_missing/558 deleted file mode 100644 index 2d6178e4a..000000000 --- a/gitlab/issues_text/target_missing/host_missing/accel_missing/558 +++ /dev/null @@ -1,57 +0,0 @@ -gtk UI interprets double/triple click as button release -Description of problem: -When using the GTK interface clicking rapidly in a down-up-down pattern, the final "down" event is erroneously followed by an immediate "up" event and the mouse device in the guest reports the pressed button as no longer being held. -Steps to reproduce: -1. Start a VM using the GTK interface. -2. Open a tool to examine guest mouse input events, such as `xev` or `yutani-test` -3. Click twice with any button, without releasing on the second click. -4. Observe erroneous 'up' event in guest. -5. Move the mouse while keeping the button pressed. -6. Observe the guest reports the button is not held. -Additional information: -GTK 3 sends an additional `GDK_2BUTTON_PRESS` event after the initial `GDK_BUTTON_PRESS` event, which QEMU is misinterpreting as a release event. I confirmed this with the addition of some logging of `button->type` in `gd_button_event`: - -``` -button = 1, type = 4 -button = 1, type = 7 -button = 1, type = 4 -button = 1, type = 7 -button = 1, type = 4 # = PRESS -button = 1, type = 5 # = 2BUTTON_PRESS -button = 1, type = 7 -button = 1, type = 4 -button = 1, type = 7 -button = 1, type = 4 -button = 1, type = 5 -button = 1, type = 7 -button = 1, type = 4 -button = 1, type = 7 -button = 1, type = 4 -button = 1, type = 7 -button = 1, type = 4 -button = 1, type = 7 -button = 1, type = 4 -button = 1, type = 5 -button = 1, type = 7 -``` - -```diff -diff --git a/ui/gtk.c b/ui/gtk.c -index cfb0728d1f..b9979f0e11 100644 ---- a/ui/gtk.c -+++ b/ui/gtk.c -@@ -925,6 +925,13 @@ static gboolean gd_button_event(GtkWidget *widget, GdkEventButton *button, - return TRUE; - } - -+ /* ignore additional events for double- and triple- press, as they are -+ * sent to us after a regular press event; otherwise we will misinterpret -+ * these as release events and eat the button! */ -+ if (button->type == GDK_2BUTTON_PRESS || button->type == GDK_3BUTTON_PRESS) { -+ return TRUE; -+ } -+ - qemu_input_queue_btn(vc->gfx.dcl.con, btn, - button->type == GDK_BUTTON_PRESS); - qemu_input_event_sync(); -``` |