[Qemu-devel] [PATCH] gtk: add zoom-to-fit to gtk options.

Gerd Hoffmann posted 1 patch 7 years, 2 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180824055120.9443-1-kraxel@redhat.com
Test docker-clang@ubuntu failed
Test checkpatch passed
There is a newer version of this series
ui/gtk.c     | 8 ++++++++
qapi/ui.json | 3 ++-
2 files changed, 10 insertions(+), 1 deletion(-)
[Qemu-devel] [PATCH] gtk: add zoom-to-fit to gtk options.
Posted by Gerd Hoffmann 7 years, 2 months ago
This allows to set the option on the command line, i.e. "-display
gtk,zoom-to-fit={on,off}", overriding the default choosen by qemu.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 ui/gtk.c     | 8 ++++++++
 qapi/ui.json | 3 ++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/ui/gtk.c b/ui/gtk.c
index 5cce6ed42d..3ddb5fe162 100644
--- a/ui/gtk.c
+++ b/ui/gtk.c
@@ -2136,6 +2136,8 @@ static GSList *gd_vc_gfx_init(GtkDisplayState *s, VirtualConsole *vc,
                               QemuConsole *con, int idx,
                               GSList *group, GtkWidget *view_menu)
 {
+    bool zoom_to_fit;
+
     vc->label = qemu_console_get_label(con);
     vc->s = s;
     vc->gfx.scale_x = 1.0;
@@ -2199,6 +2201,12 @@ static GSList *gd_vc_gfx_init(GtkDisplayState *s, VirtualConsole *vc,
     group = gd_vc_menu_init(s, vc, idx, group, view_menu);
 
     if (dpy_ui_info_supported(vc->gfx.dcl.con)) {
+        zoom_to_fit = true;
+    }
+    if (s->opts->u.gtk.has_zoom_to_fit) {
+        zoom_to_fit = s->opts->u.gtk.zoom_to_fit;
+    }
+    if (zoom_to_fit) {
         gtk_menu_item_activate(GTK_MENU_ITEM(s->zoom_fit_item));
         s->free_scale = true;
     }
diff --git a/qapi/ui.json b/qapi/ui.json
index 4ca91bb45a..bf2d0fa60e 100644
--- a/qapi/ui.json
+++ b/qapi/ui.json
@@ -1027,7 +1027,8 @@
 #
 ##
 { 'struct'  : 'DisplayGTK',
-  'data'    : { '*grab-on-hover' : 'bool' } }
+  'data'    : { '*grab-on-hover' : 'bool',
+                '*zoom-to-fit'   : 'bool'  } }
 
  ##
  # @DisplayGLMode:
-- 
2.9.3


Re: [Qemu-devel] [PATCH] gtk: add zoom-to-fit to gtk options.
Posted by Markus Armbruster 7 years, 2 months ago
Gerd Hoffmann <kraxel@redhat.com> writes:

> This allows to set the option on the command line, i.e. "-display
> gtk,zoom-to-fit={on,off}", overriding the default choosen by qemu.
>
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  ui/gtk.c     | 8 ++++++++
>  qapi/ui.json | 3 ++-
>  2 files changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/ui/gtk.c b/ui/gtk.c
> index 5cce6ed42d..3ddb5fe162 100644
> --- a/ui/gtk.c
> +++ b/ui/gtk.c
> @@ -2136,6 +2136,8 @@ static GSList *gd_vc_gfx_init(GtkDisplayState *s, VirtualConsole *vc,
>                                QemuConsole *con, int idx,
>                                GSList *group, GtkWidget *view_menu)
>  {
> +    bool zoom_to_fit;
> +
>      vc->label = qemu_console_get_label(con);
>      vc->s = s;
>      vc->gfx.scale_x = 1.0;
> @@ -2199,6 +2201,12 @@ static GSList *gd_vc_gfx_init(GtkDisplayState *s, VirtualConsole *vc,
>      group = gd_vc_menu_init(s, vc, idx, group, view_menu);
>  
>      if (dpy_ui_info_supported(vc->gfx.dcl.con)) {
> +        zoom_to_fit = true;
> +    }
> +    if (s->opts->u.gtk.has_zoom_to_fit) {
> +        zoom_to_fit = s->opts->u.gtk.zoom_to_fit;
> +    }
> +    if (zoom_to_fit) {
>          gtk_menu_item_activate(GTK_MENU_ITEM(s->zoom_fit_item));
>          s->free_scale = true;
>      }
> diff --git a/qapi/ui.json b/qapi/ui.json
> index 4ca91bb45a..bf2d0fa60e 100644
> --- a/qapi/ui.json
> +++ b/qapi/ui.json
> @@ -1027,7 +1027,8 @@
   ##
   # @DisplayGTK:
   #
   # GTK display options.
   #
   # @grab-on-hover: Grab keyboard input on mouse hover.

Missing:

 + # @zoom-to-fit: bla, bla (Since 3.1)

   #
   # Since: 2.12
>  #
>  ##
>  { 'struct'  : 'DisplayGTK',
> -  'data'    : { '*grab-on-hover' : 'bool' } }
> +  'data'    : { '*grab-on-hover' : 'bool',
> +                '*zoom-to-fit'   : 'bool'  } }
>  
>   ##
>   # @DisplayGLMode: