<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 7, 2014 at 4:43 PM, Fabiano Fidêncio <span dir="ltr"><<a href="mailto:fabiano@fidencio.org" target="_blank">fabiano@fidencio.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div id=":pd3" class="" style="overflow:hidden">Having it in spice-gtk gives the functionality for free for another<br>
clients than virt-viewer, right? I guess it's an enough good reason to<br>
have it in spice-gtk instead of virt-viewer.</div></blockquote></div><br></div><div class="gmail_extra">There are some reasons you may not want to use gtk bindings (modifier-key bindings, such as "ctrl+alt"), but in general, we should use gtk bindings, and this is what virt-viewer is doing for UI integration. So spice-gtk bindings are useless as soon as you try to ingrate properly. But having modifier-key bindings for this particular action probably makes sense. But then, why not share with the ungrab keybinding already mapping that (since there are few other binding options)?.<br><br>Getting back to the patch, disabling all inputs can be pretty annoying if you did it by accident. I was hoping for a more simple and less error-prone solution.<br><br></div><div class="gmail_extra">Christophe suggested something that seems better to me and avoid necessity for a second binding, it is detailed in <a href="https://bugs.freedesktop.org/show_bug.cgi?id=85331#c1">https://bugs.freedesktop.org/show_bug.cgi?id=85331#c1</a><br><br></div><div class="gmail_extra">-- <br><div class="gmail_signature">Marc-André Lureau</div>
</div></div>