[eog/wip/cdavis/hdy-consistency: 1/2] window: Use HdyWindow and HdyHeaderBar




commit 70ee7258798e46b6fbb6ca230c210c08372c6b13
Author: Christopher Davis <christopherdavis gnome org>
Date:   Mon Feb 7 10:32:42 2022 -0800

    window: Use HdyWindow and HdyHeaderBar
    
    Across GNOME core we've moved to using windows
    with rounded corners where possible. Apps not
    using rounded corners use Clutter, where it's
    not possible to clip them. EOG does not use
    Clutter, so we can use HdyWindow and HdyHeaderBar
    for consistency with the rest of GNOME's core
    app set.

 meson.build           |  1 +
 src/eog-application.c |  2 ++
 src/eog-window.c      | 25 ++++++++++++++-----------
 src/eog-window.h      |  5 +++--
 4 files changed, 20 insertions(+), 13 deletions(-)
---
diff --git a/meson.build b/meson.build
index baa5c1e8..c75e7e78 100644
--- a/meson.build
+++ b/meson.build
@@ -122,6 +122,7 @@ common_deps = [
   dependency('gdk-pixbuf-2.0', version: '>= 2.36.5'),
   dependency('glib-2.0', version: glib_req_version),
   dependency('gtk+-3.0', version: '>= 3.24.15'),
+  dependency('libhandy-1', version: '>= 1.5.0'),
   dependency('libpeas-1.0', version: peas_req_version),
 ]
 
diff --git a/src/eog-application.c b/src/eog-application.c
index 69a5c060..d26f0026 100644
--- a/src/eog-application.c
+++ b/src/eog-application.c
@@ -44,6 +44,7 @@
 #include <glib/gstdio.h>
 #include <glib-object.h>
 #include <gtk/gtk.h>
+#include <handy.h>
 
 #ifdef HAVE_EXEMPI
 #include <exempi/xmp.h>
@@ -278,6 +279,7 @@ eog_application_startup (GApplication *application)
        g_application_set_resource_base_path (application, "/org/gnome/eog");
        G_APPLICATION_CLASS (eog_application_parent_class)->startup (application);
 
+  hdy_init ();
 #ifdef HAVE_EXEMPI
        xmp_init();
 #endif
diff --git a/src/eog-window.c b/src/eog-window.c
index aec3ffd7..4668e14e 100644
--- a/src/eog-window.c
+++ b/src/eog-window.c
@@ -125,6 +125,7 @@ struct _EogWindowPrivate {
        EogWindowMode        mode;
        EogWindowStatus      status;
 
+        GtkWidget           *headerbar;
         GtkWidget           *overlay;
         GtkWidget           *box;
         GtkWidget           *layout;
@@ -177,7 +178,7 @@ struct _EogWindowPrivate {
 #endif
 };
 
-G_DEFINE_TYPE_WITH_PRIVATE (EogWindow, eog_window, GTK_TYPE_APPLICATION_WINDOW);
+G_DEFINE_TYPE_WITH_PRIVATE (EogWindow, eog_window, HDY_TYPE_APPLICATION_WINDOW);
 
 static void eog_window_action_toggle_fullscreen (GSimpleAction *action, GVariant *state, gpointer user_data);
 static void eog_window_run_fullscreen (EogWindow *window, gboolean slideshow);
@@ -1982,6 +1983,8 @@ update_ui_visibility (EogWindow *window)
        g_simple_action_set_state (G_SIMPLE_ACTION (action), g_variant_new_boolean (visible));
        gtk_widget_set_visible (priv->sidebar, visible);
 
+  gtk_widget_set_visible (priv->headerbar, !fullscreen_mode);
+
        if (priv->fullscreen_popup != NULL) {
                gtk_widget_hide (priv->fullscreen_popup);
        }
@@ -4206,12 +4209,12 @@ eog_window_construct_ui (EogWindow *window)
        gtk_container_add (GTK_CONTAINER (window), priv->box);
        gtk_widget_show (priv->box);
 
-       headerbar = gtk_header_bar_new ();
-       gtk_header_bar_set_show_close_button (GTK_HEADER_BAR (headerbar), TRUE);
-       gtk_header_bar_set_title (GTK_HEADER_BAR (headerbar),
+       priv->headerbar = hdy_header_bar_new ();
+       hdy_header_bar_set_show_close_button (HDY_HEADER_BAR (priv->headerbar), TRUE);
+       hdy_header_bar_set_title (HDY_HEADER_BAR (priv->headerbar),
                                  g_get_application_name ());
-       gtk_window_set_titlebar (GTK_WINDOW (window), headerbar);
-       gtk_widget_show (headerbar);
+       gtk_box_pack_start (GTK_BOX (priv->box), priv->headerbar, FALSE, FALSE, 0);
+       gtk_widget_show (priv->headerbar);
 
 #if 0
        zoom_button = gtk_toggle_button_new ();
@@ -4223,7 +4226,7 @@ eog_window_construct_ui (EogWindow *window)
        g_signal_connect (zoom_button, "toggled",
                          G_CALLBACK (eog_window_zoom_button_toggled_cb),
                          window);
-       gtk_header_bar_pack_start (GTK_HEADER_BAR (headerbar), zoom_button);
+       hdy_header_bar_pack_start (HDY_HEADER_BAR (priv->headerbar), zoom_button);
        /* disable zoom button if no image is loaded */
        g_object_bind_property (g_action_map_lookup_action (G_ACTION_MAP(window),
                                                            "zoom-normal"),
@@ -4234,7 +4237,7 @@ eog_window_construct_ui (EogWindow *window)
        priv->zoom_revealer = gtk_revealer_new ();
        gtk_revealer_set_transition_type (GTK_REVEALER (priv->zoom_revealer),
                                          GTK_REVEALER_TRANSITION_TYPE_SLIDE_RIGHT);
-       gtk_header_bar_pack_start (GTK_HEADER_BAR (headerbar),
+       hdy_header_bar_pack_start (HDY_HEADER_BAR (priv->headerbar),
                                   priv->zoom_revealer);
        gtk_widget_show (priv->zoom_revealer);
 
@@ -4265,7 +4268,7 @@ eog_window_construct_ui (EogWindow *window)
        gtk_menu_button_set_menu_model (GTK_MENU_BUTTON (menu_button),
                                        G_MENU_MODEL (builder_object));
 
-       gtk_header_bar_pack_end (GTK_HEADER_BAR (headerbar), menu_button);
+       hdy_header_bar_pack_end (HDY_HEADER_BAR (priv->headerbar), menu_button);
        gtk_widget_show (menu_button);
 
        action = G_ACTION (g_property_action_new ("toggle-gear-menu",
@@ -4279,7 +4282,7 @@ eog_window_construct_ui (EogWindow *window)
                                        "win.view-fullscreen");
        gtk_widget_set_tooltip_text(fullscreen_button,
                                    _("Show the current image in fullscreen mode"));
-       gtk_header_bar_pack_end (GTK_HEADER_BAR (headerbar), fullscreen_button);
+       hdy_header_bar_pack_end (HDY_HEADER_BAR (priv->headerbar), fullscreen_button);
        gtk_widget_show (fullscreen_button);
 
        priv->gear_menu_builder = builder;
@@ -4398,7 +4401,7 @@ eog_window_construct_ui (EogWindow *window)
        zoom_entry = eog_zoom_entry_new (EOG_SCROLL_VIEW (priv->view),
                                         G_MENU (gtk_builder_get_object (builder,
                                                                         "zoom-menu")));
-       gtk_header_bar_pack_start (GTK_HEADER_BAR (headerbar), zoom_entry);
+       hdy_header_bar_pack_start (HDY_HEADER_BAR (priv->headerbar), zoom_entry);
 
        priv->thumbview = g_object_ref (eog_thumb_view_new ());
 
diff --git a/src/eog-window.h b/src/eog-window.h
index eabd3803..ede18152 100644
--- a/src/eog-window.h
+++ b/src/eog-window.h
@@ -35,6 +35,7 @@
 #include <glib-object.h>
 #include <gio/gmenu.h>
 #include <gtk/gtk.h>
+#include <handy.h>
 
 G_BEGIN_DECLS
 
@@ -84,13 +85,13 @@ typedef enum {
 } EogStartupFlags;
 
 struct _EogWindow {
-       GtkApplicationWindow win;
+       HdyApplicationWindow win;
 
        EogWindowPrivate *priv;
 };
 
 struct _EogWindowClass {
-       GtkApplicationWindowClass parent_class;
+       HdyApplicationWindowClass parent_class;
 
        void (* prepared) (EogWindow *window);
 };


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]