[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RESEND] ui/gtk: Restore keyboard focus after Pag
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH RESEND] ui/gtk: Restore keyboard focus after Page change |
Date: |
Thu, 17 Jul 2014 14:21:48 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Jul 08, 2014 at 02:28:57PM -0400, John Snow wrote:
> (Resending for correct email addresses via MAINTAINERS ...)
>
> In the GTK UI, after changing focus to the qemu monitor Notebook Page,
> when restoring focus to the virtual machine page, the keyboard focus is lost
> to a hidden GTK widget. Focus can only be restored to the virtual machine by
> pressing "tab" or any of the four directional arrow keys.
>
> Clicking in the window or grabbing/ungrabbing input does not restore keyboard
> focus to the child widget.
>
> This patch adjusts the Notebook page switching callback to automatically
> steal keyboard focus on the Page switch event, so that keyboard input
> does not appear to break or disappear after tabbing to the QEMU monitor.
>
> Signed-off-by: John Snow <address@hidden>
> ---
> ui/gtk.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/ui/gtk.c b/ui/gtk.c
> index b02fcd6..2345d7e 100644
> --- a/ui/gtk.c
> +++ b/ui/gtk.c
> @@ -992,13 +992,16 @@ static void gd_menu_switch_vc(GtkMenuItem *item, void
> *opaque)
> {
> GtkDisplayState *s = opaque;
> VirtualConsole *vc = gd_vc_find_by_menu(s);
> + GtkNotebook *nb = GTK_NOTEBOOK(s->notebook);
> + GtkWidget *child;
> gint page;
>
> gtk_release_modifiers(s);
> if (vc) {
> - page = gtk_notebook_page_num(GTK_NOTEBOOK(s->notebook),
> - vc->tab_item);
> - gtk_notebook_set_current_page(GTK_NOTEBOOK(s->notebook), page);
> + page = gtk_notebook_page_num(nb, vc->tab_item);
> + gtk_notebook_set_current_page(nb, page);
> + child = gtk_notebook_get_nth_page(nb, page);
> + gtk_widget_grab_focus(child);
> }
> }
>
> --
> 1.9.3
Ping?
pgpdj9zYrDfJu.pgp
Description: PGP signature