On Mon, Jun 15, 2015 at 10:28:05AM +0200, Pavel Grunt wrote:
> On Thu, 2015-06-11 at 10:27 +0200, Christophe Fergeau wrote:
> > Enabling hotkeys will trigger a rebuild of the 'send keys' menu
> > containing the new hotkeys. virt_viewer_app_set_hotkeys() was clearing
> > and then enabling the hotkeys before parsing the string containing the
> > new hotkeys. This was causing these hotkeys to be missing from the 'Send
> > keys' menu when they are set through the spice controller because the
> > 'send keys' menu was rebuilt before the new hotkeys are set.
> >
> > Resolves: rhbz#1055600
> > ---
> > src/virt-viewer-app.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
> > index 8a08b6b..2bf74f7 100644
> > --- a/src/virt-viewer-app.c
> > +++ b/src/virt-viewer-app.c
> > @@ -2068,7 +2068,6 @@ virt_viewer_app_set_hotkeys(VirtViewerApp *self, const
> > gchar *hotkeys_str)
> > }
> >
> > virt_viewer_app_clear_hotkeys(self);
> > - virt_viewer_app_set_enable_accel(self, TRUE);
> >
> > for (hotkey = hotkeys; *hotkey != NULL; hotkey++) {
> > gchar *key = strstr(*hotkey, "=");
> > @@ -2100,6 +2099,7 @@ virt_viewer_app_set_hotkeys(VirtViewerApp *self, const
> > gchar *hotkeys_str)
> > }
> > g_strfreev(hotkeys);
> >
> > + virt_viewer_app_set_enable_accel(self, TRUE);
> > virt_viewer_update_smartcard_accels(self);
> > }
> >
>
> Ack,
Thanks, pushed.
Christophe
Attachment:
pgpdkC1GXm9Nh.pgp
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
