From e43919cb9e2ba4c93389c191b2c64936c25f2886 Mon Sep 17 00:00:00 2001
From: Mitchell Hashimoto <mitchell.hashimoto@gmail.com>
Date: Fri, 29 Dec 2023 09:10:49 -0800
Subject: [PATCH] apprt/gtk: stylistic changes to adwaita init

---
 src/apprt/gtk/App.zig | 56 ++++++++++++++++++++++---------------------
 1 file changed, 29 insertions(+), 27 deletions(-)

diff --git a/src/apprt/gtk/App.zig b/src/apprt/gtk/App.zig
index db7261714..f687c68a0 100644
--- a/src/apprt/gtk/App.zig
+++ b/src/apprt/gtk/App.zig
@@ -103,37 +103,39 @@ pub fn init(core_app: *CoreApp, opts: Options) !App {
     };
 
     // Create our GTK Application which encapsulates our process.
-    log.debug("creating GTK application id={s} single-instance={}", .{
-        app_id,
-        single_instance,
-    });
-
     const app: *c.GtkApplication = app: {
-        if (build_options.libadwaita and config.@"gtk-adwaita") {
-            const adw_app = @as(?*c.AdwApplication, @ptrCast(c.adw_application_new(
-                app_id.ptr,
-                app_flags,
-            ))) orelse return error.GtkInitFailed;
+        const adwaita = build_options.libadwaita and config.@"gtk-adwaita";
 
-            const style_manager = c.adw_application_get_style_manager(adw_app);
-            c.adw_style_manager_set_color_scheme(
-                style_manager,
-                switch (config.@"window-theme") {
-                    .system => c.ADW_COLOR_SCHEME_PREFER_LIGHT,
-                    .dark => c.ADW_COLOR_SCHEME_FORCE_DARK,
-                    .light => c.ADW_COLOR_SCHEME_FORCE_LIGHT,
-                },
-            );
+        log.debug("creating GTK application id={s} single-instance={} adwaita={}", .{
+            app_id,
+            single_instance,
+            adwaita,
+        });
 
-            break :app @ptrCast(adw_app);
-        } else {
-            const app = @as(?*c.GtkApplication, @ptrCast(c.gtk_application_new(
-                app_id.ptr,
-                app_flags,
-            ))) orelse return error.GtkInitFailed;
+        // If not libadwaita, create a standard GTK application.
+        if (!adwaita) break :app @as(?*c.GtkApplication, @ptrCast(c.gtk_application_new(
+            app_id.ptr,
+            app_flags,
+        ))) orelse return error.GtkInitFailed;
 
-            break :app app;
-        }
+        // Use libadwaita if requested. Using an AdwApplication lets us use
+        // Adwaita widgets and access things such as the color scheme.
+        const adw_app = @as(?*c.AdwApplication, @ptrCast(c.adw_application_new(
+            app_id.ptr,
+            app_flags,
+        ))) orelse return error.GtkInitFailed;
+
+        const style_manager = c.adw_application_get_style_manager(adw_app);
+        c.adw_style_manager_set_color_scheme(
+            style_manager,
+            switch (config.@"window-theme") {
+                .system => c.ADW_COLOR_SCHEME_PREFER_LIGHT,
+                .dark => c.ADW_COLOR_SCHEME_FORCE_DARK,
+                .light => c.ADW_COLOR_SCHEME_FORCE_LIGHT,
+            },
+        );
+
+        break :app @ptrCast(adw_app);
     };
 
     errdefer c.g_object_unref(app);