[virt-tools-list] [PATCH] hotkeys: send modifiers before non-modifier key

[Christophe Fergeau]

On Thu, Sep 26, 2013 at 04:33:12PM +0200, Marc-André Lureau wrote:
> This fix the "send menu" for hotkeys set with non-modifiers keys. The

'This fixes'

> current order of press events is wrong, as it sends first non-modifiers
> keys, and in general Ctrl+t will work, T+Ctrl will not.

I'd make the Ctrl+t / T+Ctrl consistent with respect to case.
Does this deserve a commennt in accel_key_to_keys?

ACK.

Christophe

> 
> https://bugzilla.redhat.com/show_bug.cgi?id=846006
> ---
>  src/virt-viewer-window.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/virt-viewer-window.c b/src/virt-viewer-window.c
> index 7593e3c..b70aaf8 100644
> --- a/src/virt-viewer-window.c
> +++ b/src/virt-viewer-window.c
> @@ -632,9 +632,6 @@ accel_key_to_keys(const GtkAccelKey *key)
>      guint val;
>      GArray *a = g_array_new(FALSE, FALSE, sizeof(guint));
>  
> -    val = key->accel_key;
> -    g_array_append_val(a, val);
> -
>      g_warn_if_fail((key->accel_mods &
>                      ~(GDK_SHIFT_MASK | GDK_CONTROL_MASK | GDK_MOD1_MASK)) == 0);
>  
> @@ -653,6 +650,9 @@ accel_key_to_keys(const GtkAccelKey *key)
>          g_array_append_val(a, val);
>      }
>  
> +    val = key->accel_key;
> +    g_array_append_val(a, val);
> +
>      val = GDK_VoidSymbol;
>      g_array_append_val(a, val);
>  
> -- 
> 1.8.3.1
> 
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20130926/781d6981/attachment.sig>