Fix Win tray hidden-state toggle logic

changelog.rst

@@ -1,28 +1,6 @@
 ZoiteChat ChangeLog
 =================
 
-2.18.0~pre5 (2026-03-22)
-------------------------
-- Overhauled preferences/config saving: fully staged and transactional, debounced
-  with flush on close, write failures now surfaced.
-- Overhauled GTK theme handling: live preview, correct colour reset persistence,
-  proper file:// import paths, consistent menubar CSS.
-- Fixed GTK entry scroll artifact in the input box.
-- Hardened tray menu lifetime; fixed stale pointer crash on menu destruction.
-- Fixed AppIndicator tray init for Wayland/source builds; skip redundant X11
-  tray probe for AppIndicator builds.
-- Split topic/mode rows in channel bar, tighten spacing, persist userlist column widths.
-- Preserve saved right-pane size on first layout.
-- Fixed sounds prefs section.
-- Fixed chanview tree layout and header alignment; tighten topic URL hit-testing.
-- Dropped realpath() in favour of GLib-only absolute path build.
-- Made About dialog links explicit; added GPL licence URL.
-- Added licence headers to new source files.
-- Windows installer: fixed VC++ redist URL, added non-plugin download fallback,
-  switched to registry-based runtime detection.
-- Made libayatana-appindicator a required dep in PKGBUILD.
-- Cleaned up Meson libperl detection.
-
 2.18.0~pre4 (2026-03-15)
 ------------------------
 

data/misc/net.zoite.Zoitechat.appdata.xml.in

@@ -29,45 +29,6 @@
     <id>zoitechat.desktop</id>
   </provides>
 <releases>
-  <release date="2026-03-22" version="2.18.0~pre5">
-    <description>
-      <p>Preferences and config saving:</p>
-      <ul>
-        <li>Overhauled preferences/config saving: fully staged and transactional, debounced with flush on close, write failures now surfaced.</li>
-      </ul>
-      <p>GTK theme and UI:</p>
-      <ul>
-        <li>Overhauled GTK theme handling: live preview, correct colour reset persistence, proper <code>file://</code> import paths, consistent menubar CSS.</li>
-        <li>Fixed GTK entry scroll artifact in the input box.</li>
-        <li>Split topic/mode rows in channel bar, tighten spacing, persist userlist column widths.</li>
-        <li>Preserve saved right-pane size on first layout.</li>
-        <li>Fixed sounds prefs section.</li>
-        <li>Fixed chanview tree layout and header alignment; tighten topic URL hit-testing.</li>
-      </ul>
-      <p>Tray:</p>
-      <ul>
-        <li>Hardened tray menu lifetime; fixed stale pointer crash on menu destruction.</li>
-        <li>Fixed AppIndicator tray init for Wayland/source builds; skip redundant X11 tray probe for AppIndicator builds.</li>
-      </ul>
-      <p>Build and packaging:</p>
-      <ul>
-        <li>Dropped <code>realpath()</code> in favour of GLib-only absolute path build.</li>
-        <li>Made About dialog links explicit; added GPL licence URL.</li>
-        <li>Added licence headers to new source files.</li>
-        <li>Windows installer: fixed VC++ redist URL, added non-plugin download fallback, switched to registry-based runtime detection.</li>
-        <li>Made <code>libayatana-appindicator</code> a required dep in PKGBUILD.</li>
-        <li>Cleaned up Meson libperl detection.</li>
-      </ul>
-    </description>
-  </release>
-  <release date="2026-03-22" version="2.18.0~pre5">
-    <description>
-      <p>Version metadata update:</p>
-      <ul>
-        <li>Bumped release version references to <code>2.18.0~pre5</code> across build and packaging files.</li>
-      </ul>
-    </description>
-  </release>
   <release date="2026-03-14" version="2.18.0~pre4">
     <description>
       <p>UI fixes, topic bar improvements, and selection styling updates:</p>

meson.build

@@ -1,5 +1,5 @@
 project('zoitechat', 'c',
-  version: '2.18.0~pre5',
+  version: '2.18.0~pre4',
   meson_version: '>= 0.55.0',
   default_options: [
     'c_std=c17',

plugins/python/python.py

@@ -19,7 +19,7 @@ else:
 if not hasattr(sys, 'argv'):
     sys.argv = ['<zoitechat>']
 
-VERSION = b'2.18.0~pre5'
+VERSION = b'2.18.0~pre4'
 PLUGIN_NAME = ffi.new('char[]', b'Python')
 PLUGIN_DESC = ffi.new('char[]', b'Python %d.%d scripting interface' % (sys.version_info[0], sys.version_info[1]))
 PLUGIN_VERSION = ffi.new('char[]', VERSION)

src/fe-gtk/maingui.c

@@ -4375,10 +4375,37 @@ mg_win32_filter (GdkXEvent *xevent, GdkEvent *event, gpointer data)
 			{
 				if (strcmp (command, "__WIN32_TASKBAR_TOGGLE__") == 0)
 				{
-					if (gtk_widget_get_visible (current_sess->gui->window))
-						fe_ctrl_gui (current_sess, FE_GUI_ICONIFY, 0);
+					GtkWidget *widget = current_sess->gui->window;
+					GdkWindow *gdk_window = NULL;
+					gboolean hidden = TRUE;
+
+					if (widget)
+					{
+						hidden = !gtk_widget_get_visible (widget);
+						gdk_window = gtk_widget_get_window (widget);
+						if (!hidden && gdk_window &&
+						    (gdk_window_get_state (gdk_window) & GDK_WINDOW_STATE_ICONIFIED))
+							hidden = TRUE;
+					}
+
+					if (hidden)
+					{
+						if (widget)
+						{
+							gtk_widget_show (widget);
+							gtk_window_deiconify (GTK_WINDOW (widget));
+							gtk_window_present (GTK_WINDOW (widget));
+						}
+						else
+							fe_ctrl_gui (current_sess, FE_GUI_SHOW, 0);
+					}
 					else
-						fe_ctrl_gui (current_sess, FE_GUI_SHOW, 0);
+					{
+						if (widget)
+							gtk_window_iconify (GTK_WINDOW (widget));
+						else
+							fe_ctrl_gui (current_sess, FE_GUI_ICONIFY, 0);
+					}
 				}
 				else
 				{

src/fe-gtk/plugin-tray.c

@@ -125,7 +125,6 @@ static void tray_menu_notify_cb (GObject *tray, GParamSpec *pspec, gpointer user
 static void tray_update_toggle_item_label (void);
 static gboolean tray_window_state_cb (GtkWidget *widget, GdkEventWindowState *event, gpointer userdata);
 static void tray_window_visibility_cb (GtkWidget *widget, gpointer userdata);
-static void tray_toggle_item_destroy_cb (GtkWidget *widget, gpointer userdata);
 #if HAVE_APPINDICATOR_BACKEND
 static void tray_menu_show_cb (GtkWidget *menu, gpointer userdata) G_GNUC_UNUSED;
 #endif
@@ -409,8 +408,7 @@ tray_app_indicator_cleanup (void)
 
 	if (tray_menu)
 	{
-		if (GTK_IS_WIDGET (tray_menu))
-			gtk_widget_destroy (tray_menu);
+		gtk_widget_destroy (tray_menu);
 		tray_menu = NULL;
 	}
 }
@@ -584,23 +582,33 @@ tray_backend_cleanup (void)
 	tray_backend_active = FALSE;
 }
 
+static gboolean
+tray_window_is_hidden (GtkWidget *widget)
+{
+	GdkWindow *gdk_win;
+
+	if (!widget || !gtk_widget_get_visible (widget))
+		return TRUE;
+
+	gdk_win = gtk_widget_get_window (widget);
+	if (gdk_win && (gdk_window_get_state (gdk_win) & GDK_WINDOW_STATE_ICONIFIED))
+		return TRUE;
+
+	return FALSE;
+}
+
 static WinStatus
 tray_get_window_status (void)
 {
 	GtkWindow *win;
 	GtkWidget *widget;
-	GdkWindow *gdk_win;
 	const char *st;
 
 	win = GTK_WINDOW (zoitechat_get_info (ph, "gtkwin_ptr"));
 	if (win)
 	{
 		widget = GTK_WIDGET (win);
-		if (!gtk_widget_get_visible (widget))
-			return WS_HIDDEN;
-
-		gdk_win = gtk_widget_get_window (widget);
-		if (gdk_win && (gdk_window_get_state (gdk_win) & GDK_WINDOW_STATE_ICONIFIED))
+		if (tray_window_is_hidden (widget))
 			return WS_HIDDEN;
 	}
 
@@ -855,6 +863,8 @@ tray_toggle_visibility (gboolean force_hide)
 	static int maximized;
 	static int fullscreen;
 	GtkWindow *win;
+	GtkWidget *widget;
+	gboolean hidden;
 
 	if (!tray_backend_active)
 		return FALSE;
@@ -870,7 +880,10 @@ tray_toggle_visibility (gboolean force_hide)
 	if (!win)
 		return FALSE;
 
-	if (force_hide || gtk_widget_get_visible (GTK_WIDGET (win)))
+	widget = GTK_WIDGET (win);
+	hidden = tray_window_is_hidden (widget);
+
+	if (force_hide || !hidden)
 	{
 		if (prefs.hex_gui_tray_away)
 			zoitechat_command (ph, "ALLSERV AWAY");
@@ -1028,15 +1041,10 @@ blink_item (unsigned int *setting, GtkWidget *menu, char *label)
 static void
 tray_menu_destroy (GtkWidget *menu, gpointer userdata)
 {
-	GtkWidget **menu_ptr = userdata;
+	(void)userdata;
 
-	if (menu_ptr && *menu_ptr == menu)
-		*menu_ptr = NULL;
-
-	if (GTK_IS_WIDGET (menu))
-		gtk_widget_destroy (menu);
-	if (G_IS_OBJECT (menu))
-		g_object_unref (menu);
+	gtk_widget_destroy (menu);
+	g_object_unref (menu);
 #ifdef WIN32
 	g_source_remove (tray_menu_timer);
 #endif
@@ -1096,8 +1104,6 @@ tray_menu_populate (GtkWidget *menu)
 	zoitechat_set_context (ph, zoitechat_find_context (ph, NULL, NULL));
 
 	tray_toggle_item = tray_make_item (menu, _("_Hide Window"), tray_menu_restore_cb, NULL);
-	g_signal_connect (G_OBJECT (tray_toggle_item), "destroy",
-		G_CALLBACK (tray_toggle_item_destroy_cb), NULL);
 	tray_update_toggle_item_label ();
 	tray_make_item (menu, NULL, tray_menu_quit_cb, NULL);
 
@@ -1138,22 +1144,12 @@ tray_menu_clear (GtkWidget *menu)
 
 	children = gtk_container_get_children (GTK_CONTAINER (menu));
 	for (iter = children; iter; iter = iter->next)
-		if (GTK_IS_WIDGET (iter->data))
-			gtk_widget_destroy (GTK_WIDGET (iter->data));
+		gtk_widget_destroy (GTK_WIDGET (iter->data));
 	g_list_free (children);
 	tray_toggle_item = NULL;
 }
 #endif
 
-static void
-tray_toggle_item_destroy_cb (GtkWidget *widget, gpointer userdata)
-{
-	(void)userdata;
-
-	if (tray_toggle_item == widget)
-		tray_toggle_item = NULL;
-}
-
 static void
 tray_update_toggle_item_label (void)
 {
@@ -1161,11 +1157,6 @@ tray_update_toggle_item_label (void)
 
 	if (!tray_toggle_item)
 		return;
-	if (!GTK_IS_MENU_ITEM (tray_toggle_item))
-	{
-		tray_toggle_item = NULL;
-		return;
-	}
 
 	if (tray_get_window_status () == WS_HIDDEN)
 		label = _("_Restore Window");
@@ -1218,9 +1209,10 @@ tray_menu_cb (GtkWidget *widget, guint button, guint time, gpointer userdata)
 	(void)time;
 	(void)userdata;
 
-	if (menu)
+	/* close any old menu */
+	if (G_IS_OBJECT (menu))
 	{
-		tray_menu_destroy (menu, &menu);
+		tray_menu_destroy (menu, NULL);
 	}
 
 	menu = gtk_menu_new ();
@@ -1230,9 +1222,8 @@ tray_menu_cb (GtkWidget *widget, guint button, guint time, gpointer userdata)
 	g_object_ref (menu);
 	g_object_ref_sink (menu);
 	g_object_unref (menu);
-	g_object_add_weak_pointer (G_OBJECT (menu), (gpointer *)&menu);
 	g_signal_connect (G_OBJECT (menu), "selection-done",
-		G_CALLBACK (tray_menu_destroy), &menu);
+		G_CALLBACK (tray_menu_destroy), NULL);
 #ifdef WIN32
 	g_signal_connect (G_OBJECT (menu), "leave-notify-event",
 		G_CALLBACK (tray_menu_left_cb), NULL);

src/fe-gtk/setup.c

@@ -2197,8 +2197,8 @@ setup_ok_cb (GtkWidget *but, GtkWidget *win)
         char buffer[192];
 
         memcpy (&old_prefs, &prefs, sizeof (prefs));
-        setup_apply (&setup_prefs);
         theme_preferences_stage_apply ();
+        setup_apply (&setup_prefs);
         save_result = preferences_persistence_save_all ();
         if (save_result.success)
         {

src/fe-gtk/theme/tests/test-theme-preferences-gtk3-populate.c

@@ -68,16 +68,6 @@ theme_get_color (ThemeSemanticToken token, GdkRGBA *color)
         return TRUE;
 }
 
-void
-theme_get_widget_style_values_for_widget (GtkWidget *widget, ThemeWidgetStyleValues *out_values)
-{
-        (void)widget;
-        if (!out_values)
-                return;
-        gdk_rgba_parse (&out_values->foreground, "#111111");
-        gdk_rgba_parse (&out_values->background, "#f0f0f0");
-}
-
 void
 theme_manager_set_token_color (unsigned int dark_mode, ThemeSemanticToken token, const GdkRGBA *color, gboolean *changed)
 {

src/fe-gtk/theme/theme-preferences.c

@@ -723,7 +723,6 @@ theme_preferences_manager_dialog_response_cb (GtkDialog *dialog, gint response_i
                 if (theme_preferences_stage.active)
                 {
                         ThemeSemanticToken token;
-                        ThemeWidgetStyleValues style_values;
 
                         for (token = THEME_TOKEN_MIRC_0; token < THEME_TOKEN_COUNT; token++)
                         {
@@ -734,12 +733,6 @@ theme_preferences_manager_dialog_response_cb (GtkDialog *dialog, gint response_i
                                 theme_preferences_stage.staged[token] = rgba;
                                 theme_preferences_stage.staged_valid[token] = TRUE;
                         }
-                        theme_get_widget_style_values_for_widget (GTK_WIDGET (dialog), &style_values);
-                        theme_preferences_stage.staged[THEME_TOKEN_TEXT_FOREGROUND] = style_values.foreground;
-                        theme_preferences_stage.staged_valid[THEME_TOKEN_TEXT_FOREGROUND] = TRUE;
-                        theme_preferences_stage.staged[THEME_TOKEN_TEXT_BACKGROUND] = style_values.background;
-                        theme_preferences_stage.staged_valid[THEME_TOKEN_TEXT_BACKGROUND] = TRUE;
-                        theme_preferences_stage_sync_runtime_to_staged ();
                         theme_preferences_stage_recompute_changed ();
                         if (ui->color_change_flag)
                                 *ui->color_change_flag = theme_preferences_stage.changed;

win32/version.txt

@@ -1 +1 @@
-2.18.0~pre5
+2.18.0~pre4