offline docs by default.

.github/workflows/appimage-build.yml

@@ -37,6 +37,7 @@ jobs:
             liblua5.4-dev libpci-dev libperl-dev libssl-dev libayatana-appindicator3-dev \
             perl python3 python3-minimal python3-dev python3-cffi mono-devel desktop-file-utils \
             fonts-noto-color-emoji breeze-gtk-theme \
+            python3-venv \
             patchelf file curl
 
       - name: Configure
@@ -206,6 +207,10 @@ jobs:
             export ZOITECHAT_LIBDIR="$APPDIR/usr/lib/zoitechat/plugins"
           fi
 
+          if [ -f "$APPDIR/usr/share/doc/zoitechat/html/index.html" ]; then
+            export ZOITECHAT_DOCDIR="$APPDIR/usr/share/doc/zoitechat/html"
+          fi
+
           if [ -d "$APPDIR/usr/share/glib-2.0/schemas" ]; then
             export GSETTINGS_SCHEMA_DIR="$APPDIR/usr/share/glib-2.0/schemas${GSETTINGS_SCHEMA_DIR:+:$GSETTINGS_SCHEMA_DIR}"
           fi

.github/workflows/manjaro-package-build.yml

@@ -0,0 +1,79 @@
+name: Manjaro Package Build
+
+on:
+  push:
+    branches: [master, main]
+  pull_request:
+    branches: [master, main]
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  manjaro-package:
+    runs-on: ubuntu-latest
+    container:
+      image: manjarolinux/base:latest
+
+    steps:
+      - name: Install packaging toolchain
+        run: |
+          set -euxo pipefail
+          pacman -Syu --noconfirm
+          pacman -S --noconfirm --needed \
+            base-devel \
+            git \
+            namcap \
+            meson \
+            ninja \
+            pkgconf \
+            glib2 \
+            glib2-devel \
+            gtk3 \
+            openssl \
+            libcanberra \
+            libayatana-appindicator \
+            iso-codes \
+            lua \
+            perl \
+            python \
+            python-cffi \
+            pciutils
+
+      - name: Checkout repository
+        uses: actions/checkout@v5
+        with:
+          fetch-depth: 0
+          submodules: true
+
+      - name: Build package and generate .SRCINFO (non-root)
+        run: |
+          set -euxo pipefail
+          id -u builder >/dev/null 2>&1 || useradd -m builder
+          chown -R builder:builder "$GITHUB_WORKSPACE"
+          su builder -c "cd '$GITHUB_WORKSPACE/packaging/manjaro' && makepkg --noconfirm --syncdeps --cleanbuild"
+          su builder -c "cd '$GITHUB_WORKSPACE/packaging/manjaro' && makepkg --printsrcinfo > .SRCINFO"
+
+      - name: Lint PKGBUILD and package with namcap
+        run: |
+          set -euxo pipefail
+          cd "$GITHUB_WORKSPACE/packaging/manjaro"
+          namcap PKGBUILD
+          pkg_file="$(find . -maxdepth 1 -type f -name '*.pkg.tar.*' | head -n1)"
+          namcap "$pkg_file"
+
+      - name: Collect artifacts
+        run: |
+          set -euxo pipefail
+          mkdir -p artifacts
+          cp -v "$GITHUB_WORKSPACE"/packaging/manjaro/*.pkg.tar.* artifacts/
+          cp -v "$GITHUB_WORKSPACE"/packaging/manjaro/PKGBUILD artifacts/
+          cp -v "$GITHUB_WORKSPACE"/packaging/manjaro/.SRCINFO artifacts/
+
+      - name: Upload package artifacts
+        uses: actions/upload-artifact@v6
+        with:
+          name: zoitechat-manjaro-package
+          path: artifacts/*
+          if-no-files-found: error

.github/workflows/windows-build.yml

@@ -113,6 +113,13 @@ jobs:
             New-Item -Path $pyDir -Name "${{ matrix.platform }}" -ItemType Junction -Value $pyRoot | Out-Null
           }
 
+      # Runs while Python 3.14 is still the active interpreter; the installer
+      # packages rel\offline-docs as the "Offline Documentation" component and
+      # skips it if this step could not build the docs.
+      - name: Build offline documentation
+        run: |
+          python data\misc\build_offline_docs.py --output ..\zoitechat-build\${{ matrix.platform }}\rel\offline-docs
+
       - uses: actions/setup-python@v6
         with:
           python-version: '3.8.10'

data/meson.build

@@ -6,4 +6,29 @@ if get_option('gtk-frontend')
   subdir('icons')
   subdir('misc')
   subdir('man')
+
+  # Fetch and build the offline manual so Help->Contents works without
+  # connectivity. 'auto' skips with a warning when the docs cannot be
+  # fetched (e.g. network-less distro builds, which ship docs themselves);
+  # 'enabled' makes that a hard error.
+  offline_docs_opt = get_option('offline-docs')
+  if not offline_docs_opt.disabled()
+    offline_docs_args = [
+      '--repo', get_option('offline-docs-repo'),
+      '--output', join_paths(meson.current_build_dir(), 'offline-docs'),
+      '--stamp', '@OUTPUT@',
+    ]
+    if offline_docs_opt.enabled()
+      offline_docs_args += ['--strict']
+    endif
+    custom_target('offline-docs',
+      output: 'offline-docs.stamp',
+      command: [find_program('python3'), files('misc/build_offline_docs.py')] + offline_docs_args,
+      build_by_default: true,
+    )
+    meson.add_install_script(files('misc/install_offline_docs.py'),
+      join_paths(meson.current_build_dir(), 'offline-docs'),
+      offline_docs_dir,
+    )
+  endif
 endif

data/misc/build_offline_docs.py

@@ -0,0 +1,133 @@
+#!/usr/bin/env python3
+"""Build ZoiteChat's offline HTML documentation for bundling with releases.
+
+Clones the ZoiteChat/documentation repository, preferring a tag that
+matches this source tree's version so the bundled manual matches the
+application, and builds it with Sphinx into --output. Used by the
+meson build (offline-docs option) and by the Windows CI workflow.
+
+Sphinx is installed into a throwaway virtualenv so the invoking Python
+(which the Windows build also bundles pieces of) is left untouched.
+
+Unless --strict is given, any failure prints a warning (GitHub Actions
+annotation syntax) and exits 0: builds then simply fall back to opening
+the online manual instead of failing outright.
+"""
+
+import argparse
+import pathlib
+import re
+import shutil
+import subprocess
+import sys
+import tempfile
+
+DEFAULT_REPO = "https://github.com/ZoiteChat/documentation.git"
+
+
+def project_version() -> str:
+    meson_build = pathlib.Path(__file__).resolve().parents[2] / "meson.build"
+    match = re.search(r"version\s*:\s*'([^']+)'", meson_build.read_text(encoding="utf-8"))
+    if not match:
+        raise RuntimeError(f"could not parse project version from {meson_build}")
+    return match.group(1)
+
+
+def run(argv, **kwargs) -> None:
+    print("+", " ".join(str(arg) for arg in argv), flush=True)
+    subprocess.run(argv, check=True, **kwargs)
+
+
+def clone_docs(repo: str, version: str, target: pathlib.Path) -> str:
+    # A missing tag falls through to the default branch so the docs may be
+    # newer than the app, which beats shipping none at all.
+    for ref in (f"v{version}", version, None):
+        cmd = ["git", "clone", "--depth", "1"]
+        if ref:
+            cmd += ["--branch", ref]
+        cmd += [repo, str(target)]
+        try:
+            run(cmd)
+            return ref or "default branch"
+        except subprocess.CalledProcessError:
+            shutil.rmtree(target, ignore_errors=True)
+    raise RuntimeError(f"could not clone {repo}")
+
+
+def find_source_dir(checkout: pathlib.Path) -> pathlib.Path:
+    for candidate in (checkout, checkout / "docs", checkout / "doc"):
+        if (candidate / "conf.py").is_file():
+            return candidate
+    raise RuntimeError(f"no Sphinx conf.py found in {checkout}")
+
+
+def venv_python(venv_dir: pathlib.Path) -> pathlib.Path:
+    if sys.platform == "win32":
+        return venv_dir / "Scripts" / "python.exe"
+    return venv_dir / "bin" / "python"
+
+
+def build(output: pathlib.Path, repo: str, version: str) -> None:
+    with tempfile.TemporaryDirectory(prefix="zoitechat-docs-") as tmp:
+        tmp_dir = pathlib.Path(tmp)
+        checkout = tmp_dir / "documentation"
+        html_dir = tmp_dir / "html"
+        venv_dir = tmp_dir / "venv"
+
+        ref = clone_docs(repo, version, checkout)
+        print(f"building documentation from {ref}", flush=True)
+        source_dir = find_source_dir(checkout)
+
+        run([sys.executable, "-m", "venv", str(venv_dir)])
+        python = venv_python(venv_dir)
+        requirements = source_dir / "requirements.txt"
+        if not requirements.is_file():
+            requirements = checkout / "requirements.txt"
+        if requirements.is_file():
+            run([python, "-m", "pip", "install", "--quiet", "-r", str(requirements)])
+            run([python, "-m", "pip", "install", "--quiet", "sphinx"])
+        else:
+            run([python, "-m", "pip", "install", "--quiet", "sphinx", "sphinx_rtd_theme"])
+
+        run([python, "-m", "sphinx", "-b", "html", str(source_dir), str(html_dir)])
+        if not (html_dir / "index.html").is_file():
+            raise RuntimeError("Sphinx build produced no index.html")
+
+        if output.exists():
+            shutil.rmtree(output)
+        shutil.copytree(html_dir, output,
+                        ignore=shutil.ignore_patterns(".doctrees", ".buildinfo"))
+
+    file_count = sum(1 for path in output.rglob("*") if path.is_file())
+    print(f"offline documentation staged at {output} ({file_count} files)", flush=True)
+
+
+def main() -> int:
+    parser = argparse.ArgumentParser(description=__doc__)
+    parser.add_argument("--output", required=True, type=pathlib.Path,
+                        help="directory to stage the built HTML tree into")
+    parser.add_argument("--repo", default=DEFAULT_REPO,
+                        help="documentation git repository to build from")
+    parser.add_argument("--version", default=None,
+                        help="app version used to pick a matching docs tag "
+                             "(default: parsed from meson.build)")
+    parser.add_argument("--strict", action="store_true",
+                        help="fail on error instead of emitting a warning")
+    parser.add_argument("--stamp", type=pathlib.Path, default=None,
+                        help="file to touch on completion (meson target output)")
+    args = parser.parse_args()
+
+    try:
+        build(args.output, args.repo, args.version or project_version())
+    except Exception as error:  # noqa: BLE001 - degrade to website-only help
+        if args.strict:
+            raise
+        print(f"::warning::offline documentation not bundled: {error}", flush=True)
+
+    if args.stamp:
+        args.stamp.write_text("ok\n", encoding="utf-8")
+    return 0
+
+
+if __name__ == "__main__":
+    sys.exit(main())

data/misc/install_offline_docs.py

@@ -0,0 +1,31 @@
+#!/usr/bin/env python3
+"""Meson install helper: copy the built offline docs into the install tree.
+
+Silently skips when the docs were not built (offline-docs=auto without
+network); Help->Contents then falls back to the online manual.
+"""
+
+import os
+import pathlib
+import shutil
+import sys
+
+src = pathlib.Path(sys.argv[1])
+dest_arg = pathlib.PurePath(sys.argv[2])
+
+if not (src / "index.html").is_file():
+    print("offline documentation was not built; skipping install")
+    sys.exit(0)
+
+if dest_arg.is_absolute():
+    # An absolute offline-docs-dir lands outside the prefix; honour DESTDIR
+    destdir = os.environ.get("DESTDIR", "")
+    dest = pathlib.Path(destdir + str(dest_arg)) if destdir else pathlib.Path(dest_arg)
+else:
+    dest = pathlib.Path(os.environ["MESON_INSTALL_DESTDIR_PREFIX"]) / dest_arg
+
+if dest.exists():
+    shutil.rmtree(dest)
+dest.parent.mkdir(parents=True, exist_ok=True)
+shutil.copytree(src, dest)
+print(f"Installed offline documentation to {dest}")

meson.build

@@ -35,6 +35,16 @@ config_h.set_quoted('PACKAGE_NAME', meson.project_name())
 config_h.set_quoted('GETTEXT_PACKAGE', 'zoitechat')
 config_h.set_quoted('LOCALEDIR', join_paths(get_option('prefix'),
                                  get_option('datadir'), 'locale'))
+
+# Where Help->Contents looks for locally installed documentation before
+# falling back to the online manual. Packagers that ship the HTML docs
+# elsewhere can point this at their doc directory.
+offline_docs_dir = get_option('offline-docs-dir')
+if offline_docs_dir == ''
+  offline_docs_dir = join_paths(get_option('datadir'), 'doc', 'zoitechat', 'html')
+endif
+config_h.set_quoted('ZOITECHATDOCDIR', join_paths(get_option('prefix'), offline_docs_dir))
+
 config_h.set10('ENABLE_NLS', true)
 
 # Optional features

meson_options.txt

@@ -33,6 +33,15 @@ option('install-appdata', type: 'boolean',
 option('install-plugin-metainfo', type: 'boolean', value: false,
   description: 'Installs metainfo files for enabled plugins, useful when distros create split packages'
 )
+option('offline-docs', type: 'feature', value: 'auto',
+  description: 'Fetch, build and install the offline HTML documentation (needs network at build time); auto skips with a warning when unavailable'
+)
+option('offline-docs-dir', type: 'string', value: '',
+  description: 'Directory searched for offline HTML documentation (index.html), absolute or relative to prefix. Defaults to $datadir/doc/zoitechat/html'
+)
+option('offline-docs-repo', type: 'string', value: 'https://github.com/ZoiteChat/documentation.git',
+  description: 'Git repository the offline documentation is built from'
+)
 
 # Plugins
 option('with-checksum', type: 'boolean',

packaging/manjaro/PKGBUILD

@@ -0,0 +1,69 @@
+# Maintainer: ZoiteChat Maintainers <zoitechat@users.noreply.github.com>
+
+pkgname=zoitechat-git
+pkgver=0
+pkgrel=1
+pkgdesc='IRC client for GNOME (ZoiteChat development snapshot)'
+arch=('x86_64')
+url='https://github.com/zoitechat/zoitechat'
+license=('GPL-2.0-or-later')
+depends=(
+  'glib2'
+  'gtk3'
+  'iso-codes'
+  'libayatana-appindicator'
+  'libcanberra'
+  'lua'
+  'openssl'
+  'perl'
+  'python-cffi'
+)
+makedepends=(
+  'git'
+  'meson'
+  'ninja'
+  'pkgconf'
+  'python'
+)
+optdepends=(
+  'pciutils: sysinfo plugin hardware detection details'
+)
+provides=('zoitechat')
+conflicts=('zoitechat')
+
+_repo_root="$(cd "$(dirname "${BASH_SOURCE[0]}")/../.." && pwd)"
+source=("zoitechat::git+file://${_repo_root}")
+sha256sums=('SKIP')
+
+pkgver() {
+  cd "$srcdir/zoitechat"
+
+  if git describe --long --tags --abbrev=7 >/dev/null 2>&1; then
+    git describe --long --tags --abbrev=7 \
+      | sed 's/^v//' \
+      | sed -E 's/([^-]+)-([0-9]+)-g/\1.r\2.g/' \
+      | sed 's/-/./g'
+  else
+    printf '0.r%s.%s\n' \
+      "$(git rev-list --count HEAD)" \
+      "$(git rev-parse --short=7 HEAD)"
+  fi
+}
+
+build() {
+  arch-meson zoitechat build \
+    -Dtext-frontend=false \
+    -Dwith-checksum=true \
+    -Dwith-fishlim=true \
+    -Dwith-lua=lua \
+    -Dwith-perl=perl \
+    -Dwith-python=python3 \
+    -Dwith-sysinfo=true \
+    -Dinstall-plugin-metainfo=true
+
+  meson compile -C build
+}
+
+package() {
+  meson install -C build --destdir "$pkgdir"
+}

src/common/fe.h

@@ -142,7 +142,6 @@ void fe_set_away (server *serv);
 void fe_serverlist_open (session *sess);
 void fe_get_bool (char *title, char *prompt, void *callback, void *userdata);
 void fe_get_str (char *prompt, char *def, void *callback, void *ud);
-void fe_get_password (char *prompt, void *callback, void *ud);
 void fe_get_int (char *prompt, int def, void *callback, void *ud);
 #define FRF_WRITE 1				/* save file */
 #define FRF_MULTIPLE 2			/* multi-select */

src/common/gtk3-theme-service.c

@@ -315,59 +315,6 @@ resolve_parent_theme_root (const char *child_theme_root, const char *parent_name
 		return home_local;
 	g_free (home_local);
 
-	{
-		const char *xdg_data_home = g_getenv ("XDG_DATA_HOME");
-
-		if (xdg_data_home && xdg_data_home[0])
-		{
-			candidate = g_build_filename (xdg_data_home, "themes", parent_name, NULL);
-			if (path_exists_as_dir (candidate))
-				return candidate;
-			g_free (candidate);
-		}
-	}
-
-	candidate = g_build_filename (g_get_user_data_dir (), "themes", parent_name, NULL);
-	if (path_exists_as_dir (candidate))
-		return candidate;
-	g_free (candidate);
-
-	{
-		const char *xdg_data_dirs = g_getenv ("XDG_DATA_DIRS");
-		guint i;
-
-		if (xdg_data_dirs && xdg_data_dirs[0])
-		{
-			char **data_dirs = g_strsplit (xdg_data_dirs, G_SEARCHPATH_SEPARATOR_S, -1);
-
-			for (i = 0; data_dirs && data_dirs[i]; i++)
-			{
-				if (!data_dirs[i][0])
-					continue;
-				candidate = g_build_filename (data_dirs[i], "themes", parent_name, NULL);
-				if (path_exists_as_dir (candidate))
-				{
-					g_strfreev (data_dirs);
-					return candidate;
-				}
-				g_free (candidate);
-			}
-			g_strfreev (data_dirs);
-		}
-		else
-		{
-			const char *const *system_data_dirs = g_get_system_data_dirs ();
-
-			for (i = 0; system_data_dirs && system_data_dirs[i]; i++)
-			{
-				candidate = g_build_filename (system_data_dirs[i], "themes", parent_name, NULL);
-				if (path_exists_as_dir (candidate))
-					return candidate;
-				g_free (candidate);
-			}
-		}
-	}
-
 	user_dir = zoitechat_gtk3_theme_service_get_user_themes_dir ();
 	candidate = g_build_filename (user_dir, parent_name, NULL);
 	g_free (user_dir);
@@ -685,47 +632,12 @@ zoitechat_gtk3_theme_service_discover (void)
 
 	g_mkdir_with_parents (user_dir, 0700);
 
-	/* Prefer the live XDG environment over GLib's process-cached copies so
-	 * theme lookup matches what GTK resolves for the running session. */
-	{
-		const char *xdg_data_home = g_getenv ("XDG_DATA_HOME");
-
-		if (xdg_data_home && xdg_data_home[0])
-			user_data_themes = g_build_filename (xdg_data_home, "themes", NULL);
-		else
 	user_data_themes = g_build_filename (g_get_user_data_dir (), "themes", NULL);
-	}
-	/* Scan the app's own themes dir first: imported themes get a symlink
-	 * under the XDG data themes dir so GTK can resolve gtk-theme-name, and
-	 * scanning that link first would make the canonical-root dedup keep
-	 * the symlink path - changing the theme's path-derived id and breaking
-	 * the saved gui_gtk3_theme reference on the next start. */
-	add_theme_root (user_roots, seen_user_roots, user_dir);
 	add_theme_root (user_roots, seen_user_roots, user_data_themes);
 	g_free (user_data_themes);
 	add_theme_root (user_roots, seen_user_roots, home_themes);
+	add_theme_root (user_roots, seen_user_roots, user_dir);
 
-	{
-		const char *xdg_data_dirs = g_getenv ("XDG_DATA_DIRS");
-
-		if (xdg_data_dirs && xdg_data_dirs[0])
-		{
-			char **data_dirs = g_strsplit (xdg_data_dirs, G_SEARCHPATH_SEPARATOR_S, -1);
-
-			for (i = 0; data_dirs && data_dirs[i]; i++)
-			{
-				char *system_themes;
-
-				if (!data_dirs[i][0])
-					continue;
-				system_themes = g_build_filename (data_dirs[i], "themes", NULL);
-				add_theme_root (system_roots, seen_system_roots, system_themes);
-				g_free (system_themes);
-			}
-			g_strfreev (data_dirs);
-		}
-		else
-		{
 	system_data_dirs = g_get_system_data_dirs ();
 	for (i = 0; system_data_dirs && system_data_dirs[i]; i++)
 	{
@@ -733,8 +645,6 @@ zoitechat_gtk3_theme_service_discover (void)
 		add_theme_root (system_roots, seen_system_roots, system_themes);
 		g_free (system_themes);
 	}
-		}
-	}
 
 	for (i = 0; i < system_roots->len; i++)
 		discover_dir (themes, seen_theme_roots, g_ptr_array_index (system_roots, i), ZOITECHAT_GTK3_THEME_SOURCE_SYSTEM);
@@ -1012,7 +922,7 @@ static gboolean
 validate_theme_root_for_import (const char *theme_root, GError **error)
 {
 	char *index_theme;
-	GKeyFile *keyfile = NULL;
+	GKeyFile *keyfile;
 	char *css_dir;
 	char *css_path;
 	char *raw_inherits;
@@ -1020,12 +930,16 @@ validate_theme_root_for_import (const char *theme_root, GError **error)
 	guint i;
 	GError *load_error = NULL;
 
-	/* index.theme is optional for GTK3 CSS themes (GTK only needs
-	 * gtk-3.x/gtk.css), but when it is present it must be well formed so
-	 * broken archives are rejected up front. */
 	index_theme = g_build_filename (theme_root, "index.theme", NULL);
-	if (g_file_test (index_theme, G_FILE_TEST_IS_REGULAR))
+	if (!g_file_test (index_theme, G_FILE_TEST_IS_REGULAR))
 	{
+		g_set_error (error, G_FILE_ERROR, G_FILE_ERROR_INVAL,
+					 "Invalid GTK3 theme at '%s': missing required index.theme at '%s'.",
+			   theme_root, index_theme);
+		g_free (index_theme);
+		return FALSE;
+	}
+
 	keyfile = g_key_file_new ();
 	if (!g_key_file_load_from_file (keyfile, index_theme, G_KEY_FILE_NONE, &load_error))
 	{
@@ -1047,7 +961,6 @@ validate_theme_root_for_import (const char *theme_root, GError **error)
 		g_free (index_theme);
 		return FALSE;
 	}
-	}
 
 	css_dir = zoitechat_gtk3_theme_pick_css_dir (theme_root);
 	if (!css_dir)
@@ -1055,7 +968,6 @@ validate_theme_root_for_import (const char *theme_root, GError **error)
 		g_set_error (error, G_FILE_ERROR, G_FILE_ERROR_INVAL,
 					 "Invalid GTK3 theme at '%s': could not resolve a GTK CSS directory (expected gtk-3.x/gtk.css).",
 					 theme_root);
-		if (keyfile)
 		g_key_file_unref (keyfile);
 		g_free (index_theme);
 		return FALSE;
@@ -1069,7 +981,6 @@ validate_theme_root_for_import (const char *theme_root, GError **error)
 			   theme_root, css_path);
 		g_free (css_path);
 		g_free (css_dir);
-		if (keyfile)
 		g_key_file_unref (keyfile);
 		g_free (index_theme);
 		return FALSE;
@@ -1077,12 +988,8 @@ validate_theme_root_for_import (const char *theme_root, GError **error)
 	g_free (css_path);
 	g_free (css_dir);
 
-	raw_inherits = NULL;
-	if (keyfile)
-	{
 	raw_inherits = g_key_file_get_string (keyfile, "Desktop Entry", "Inherits", NULL);
 	g_key_file_unref (keyfile);
-	}
 	g_free (index_theme);
 	if (!raw_inherits)
 		return TRUE;

src/common/server.c

@@ -109,58 +109,6 @@ static void server_disconnect (session * sess, int sendquit, int err);
 static int server_cleanup (server * serv);
 static void server_connect (server *serv, char *hostname, int port, int no_login);
 
-typedef struct
-{
-	server *serv;
-	char hostname[128];
-	int port;
-} reconnect_password_prompt;
-
-static gboolean
-server_login_uses_password_sasl (int logintype)
-{
-	return logintype == LOGIN_SASL ||
-		logintype == LOGIN_SASL_SCRAM_SHA_1 ||
-		logintype == LOGIN_SASL_SCRAM_SHA_256 ||
-		logintype == LOGIN_SASL_SCRAM_SHA_512;
-}
-
-static void
-server_reconnect_password_cb (int cancel, char *text, void *userdata)
-{
-	reconnect_password_prompt *prompt = userdata;
-
-	if (!cancel && is_server (prompt->serv) && !prompt->serv->connected && !prompt->serv->connecting && prompt->serv->server_session)
-	{
-		prompt->serv->password[0] = 0;
-		if (text && *text)
-			safe_strcpy (prompt->serv->password, text, sizeof (prompt->serv->password));
-		server_connect (prompt->serv, prompt->hostname, prompt->port, FALSE);
-	}
-
-	g_free (prompt);
-}
-
-static gboolean
-server_prompt_reconnect_password (server *serv)
-{
-	reconnect_password_prompt *prompt;
-	char *msg;
-	ircnet *net = serv->network;
-
-	if (!net || !(net->flags & FLAG_PROMPT_PASSWORD) || !server_login_uses_password_sasl (serv->loginmethod))
-		return FALSE;
-
-	prompt = g_new0 (reconnect_password_prompt, 1);
-	prompt->serv = serv;
-	safe_strcpy (prompt->hostname, serv->hostname, sizeof (prompt->hostname));
-	prompt->port = serv->port;
-	msg = g_strdup_printf (_("Enter SASL password for %s:"), net->name ? net->name : _("this network"));
-	fe_get_password (msg, server_reconnect_password_cb, prompt);
-	g_free (msg);
-	return TRUE;
-}
-
 static void
 write_error (char *message, GError **error)
 {
@@ -786,7 +734,6 @@ timeout_auto_reconnect (server *serv)
 		serv->recondelay_tag = 0;
 		if (!serv->connected && !serv->connecting && serv->server_session)
 		{
-			if (!server_prompt_reconnect_password (serv))
 			server_connect (serv, serv->hostname, serv->port, FALSE);
 		}
 	}

src/common/servlist.c

@@ -439,99 +439,8 @@ servlist_favchan_copy (favchannel *fav)
 	return newfav;
 }
 
-static void servlist_connect_with_password (session *sess, ircnet *net, gboolean join, const char *password);
-
-typedef struct
-{
-	session *sess;
-	ircnet *net;
-	gboolean join;
-} servlist_password_prompt;
-
-static gboolean
-servlist_login_uses_password_sasl (int logintype)
-{
-	return logintype == LOGIN_SASL ||
-		logintype == LOGIN_SASL_SCRAM_SHA_1 ||
-		logintype == LOGIN_SASL_SCRAM_SHA_256 ||
-		logintype == LOGIN_SASL_SCRAM_SHA_512;
-}
-
-static void servlist_prompt_sasl_password (servlist_password_prompt *prompt);
-
-static void
-servlist_connect_without_password_cb (int value, void *userdata)
-{
-	servlist_password_prompt *prompt = userdata;
-
-	if (value)
-	{
-		servlist_connect_with_password (prompt->sess, prompt->net, prompt->join, NULL);
-		g_free (prompt);
-	}
-	else
-	{
-		servlist_prompt_sasl_password (prompt);
-	}
-}
-
-static void
-servlist_connect_password_cb (int cancel, char *text, void *userdata)
-{
-	servlist_password_prompt *prompt = userdata;
-	char *msg;
-
-	if (cancel)
-	{
-		fe_serverlist_open (prompt->sess);
-		g_free (prompt);
-		return;
-	}
-
-	if (text && *text)
-	{
-		servlist_connect_with_password (prompt->sess, prompt->net, prompt->join, text);
-		g_free (prompt);
-		return;
-	}
-
-	msg = g_strdup_printf (_("Continue connecting to %s without a SASL password?"), prompt->net->name ? prompt->net->name : _("this network"));
-	fe_get_bool (_("No SASL password"), msg, servlist_connect_without_password_cb, prompt);
-	g_free (msg);
-}
-
-static void
-servlist_prompt_sasl_password (servlist_password_prompt *prompt)
-{
-	char *msg;
-
-	msg = g_strdup_printf (_("Enter SASL password for %s:"), prompt->net->name ? prompt->net->name : _("this network"));
-	fe_get_password (msg, servlist_connect_password_cb, prompt);
-	g_free (msg);
-}
-
 void
 servlist_connect (session *sess, ircnet *net, gboolean join)
-{
-	int logintype;
-	servlist_password_prompt *prompt;
-
-	logintype = net && net->logintype ? net->logintype : LOGIN_DEFAULT_REAL;
-	if (net && (net->flags & FLAG_PROMPT_PASSWORD) && servlist_login_uses_password_sasl (logintype))
-	{
-		prompt = g_new0 (servlist_password_prompt, 1);
-		prompt->sess = sess;
-		prompt->net = net;
-		prompt->join = join;
-		servlist_prompt_sasl_password (prompt);
-		return;
-	}
-
-	servlist_connect_with_password (sess, net, join, NULL);
-}
-
-static void
-servlist_connect_with_password (session *sess, ircnet *net, gboolean join, const char *password)
 {
 	ircserver *ircserv;
 	GSList *list;
@@ -578,11 +487,7 @@ servlist_connect_with_password (session *sess, ircnet *net, gboolean join, const
 	}
 
 	serv->password[0] = 0;
-	if (password && *password)
+	if ((net->flags & FLAG_USE_KEYRING) && net->name)
-	{
-		safe_strcpy (serv->password, password, sizeof (serv->password));
-	}
-	else if ((net->flags & FLAG_USE_KEYRING) && net->name)
 	{
 		char *stored_pass = secretstore_get_network_password (net->name);
 		if (stored_pass && *stored_pass)

src/common/servlist.h

@@ -63,8 +63,7 @@ extern GSList *network_list;
 #define FLAG_ALLOW_INVALID		32
 #define FLAG_FAVORITE			64
 #define FLAG_USE_KEYRING		128
-#define FLAG_PROMPT_PASSWORD	256
+#define FLAG_COUNT				8
-#define FLAG_COUNT				9
 
 /* Login methods. Use server password by default - if we had a NickServ password, it'd be set to 2 already by servlist_load() */
 #define LOGIN_DEFAULT_REAL		LOGIN_PASS		/* this is to set the default login method for unknown servers */

src/common/tests/test-gtk3-theme-service.c

@@ -381,10 +381,9 @@ test_archive_without_theme_reports_css_error (void)
 	}
 	archive_path = g_build_filename (tmp_root, "invalid-theme.zip", NULL);
 
-	command = g_strdup_printf ("sh -c \"cd '%s' && zip -qr '%s' .\"", archive_root, archive_path);
+	command = g_strdup_printf ("cd %s && zip -qr %s .", archive_root, archive_path);
 	g_assert_true (g_spawn_command_line_sync (command, NULL, NULL, NULL, NULL));
 	g_free (command);
-	g_assert_true (g_file_test (archive_path, G_FILE_TEST_IS_REGULAR));
 
 	g_assert_false (zoitechat_gtk3_theme_service_import (archive_path, &imported_id, &error));
 	g_assert_null (imported_id);
@@ -398,31 +397,25 @@ test_archive_without_theme_reports_css_error (void)
 }
 
 static void
-test_import_accepts_theme_missing_index_theme (void)
+test_import_rejects_theme_missing_index_theme (void)
 {
 	char *tmp_root;
 	char *src_root;
 	char *theme_root;
 	char *imported_id = NULL;
 	GError *error = NULL;
-	ZoitechatGtk3Theme *found;
 
 	setup_test_xdir (&tmp_root);
 	src_root = g_build_filename (tmp_root, "src", NULL);
 	g_mkdir_with_parents (src_root, 0700);
 	theme_root = make_theme_dir (src_root, "NoIndex", FALSE, FALSE);
 
-	/* index.theme is optional for GTK3 CSS themes: gtk-3.x/gtk.css alone
+	g_assert_false (zoitechat_gtk3_theme_service_import (theme_root, &imported_id, &error));
-	 * makes a loadable theme. */
+	g_assert_null (imported_id);
-	g_assert_true (zoitechat_gtk3_theme_service_import (theme_root, &imported_id, &error));
+	g_assert_error (error, G_FILE_ERROR, G_FILE_ERROR_INVAL);
-	g_assert_no_error (error);
+	g_assert_nonnull (g_strstr_len (error->message, -1, "missing required index.theme"));
-	g_assert_nonnull (imported_id);
+	g_assert_nonnull (g_strstr_len (error->message, -1, "NoIndex"));
-
+	g_error_free (error);
-	found = zoitechat_gtk3_theme_find_by_id (imported_id);
-	g_assert_nonnull (found);
-	g_assert_true (g_str_has_suffix (found->path, "NoIndex"));
-	zoitechat_gtk3_theme_free (found);
-	g_free (imported_id);
 
 	g_free (theme_root);
 	g_free (src_root);
@@ -670,10 +663,9 @@ test_zip_import_nested_root (void)
 	theme = make_theme_dir (nested, "Juno-ocean", TRUE, FALSE);
 	archive_path = g_build_filename (tmp_root, "themes.zip", NULL);
 
-	command = g_strdup_printf ("sh -c \"cd '%s' && zip -qr '%s' .\"", zip_root, archive_path);
+	command = g_strdup_printf ("cd %s && zip -qr %s .", zip_root, archive_path);
 	g_assert_true (g_spawn_command_line_sync (command, NULL, NULL, NULL, NULL));
 	g_free (command);
-	g_assert_true (g_file_test (archive_path, G_FILE_TEST_IS_REGULAR));
 
 	g_assert_true (zoitechat_gtk3_theme_service_import (archive_path, &imported_id, NULL));
 	found = zoitechat_gtk3_theme_find_by_id (imported_id);
@@ -705,7 +697,7 @@ main (int argc, char **argv)
 	g_test_add_func ("/gtk3-theme-service/zip-import-nested-root", test_zip_import_nested_root);
 	g_test_add_func ("/gtk3-theme-service/invalid-archive-extract-error", test_invalid_archive_reports_extract_error);
 	g_test_add_func ("/gtk3-theme-service/archive-without-theme-css-error", test_archive_without_theme_reports_css_error);
-	g_test_add_func ("/gtk3-theme-service/import-missing-index-theme", test_import_accepts_theme_missing_index_theme);
+	g_test_add_func ("/gtk3-theme-service/import-missing-index-theme", test_import_rejects_theme_missing_index_theme);
 	g_test_add_func ("/gtk3-theme-service/import-missing-desktop-entry", test_import_rejects_index_without_desktop_entry);
 	g_test_add_func ("/gtk3-theme-service/import-unresolved-inherits", test_import_rejects_unresolved_inherits);
 	return g_test_run ();

src/fe-gtk/chanview.c

@@ -174,7 +174,7 @@ chanview_apply_theme (chanview *cv)
 	chanview_set_font_desc (cv, font);
 
 	theme_manager_apply_channel_tree_style (w,
-			theme_manager_get_channel_tree_palette_behavior (cv->font_desc));
+			theme_manager_get_channel_tree_palette_behavior (font));
 }
 
 static void
@@ -307,109 +307,6 @@ chanview_set_impl (chanview *cv, int type)
 		cv->func_focus (cv->focused);
 }
 
-void
-chanview_set_sorted (chanview *cv, gboolean sorted)
-{
-	cv->sorted = sorted ? 1 : 0;
-}
-
-void
-chanview_set_use_icons (chanview *cv, gboolean use_icons)
-{
-	cv->use_icons = use_icons ? 1 : 0;
-}
-
-void
-chanview_set_trunc_len (chanview *cv, int trunc_len)
-{
-	cv->trunc_len = trunc_len;
-}
-
-typedef struct
-{
-	chan *ch;
-	int old_pos;
-} chanview_sort_entry;
-
-/* re-sort the session rows below one parent with cb_compare, leaving
- * utility rows (tag != 0) in place; mirrors chanview_insert_sorted() */
-static void
-chanview_resort_children (chanview *cv, GtkTreeIter *parent)
-{
-	GtkTreeModel *model = GTK_TREE_MODEL (cv->store);
-	GtkTreeIter iter;
-	chanview_sort_entry *entries;
-	int *positions;
-	gint *new_order;
-	chan *ch;
-	int n, i, j, count;
-
-	n = gtk_tree_model_iter_n_children (model, parent);
-	if (n < 2)
-		return;
-
-	entries = g_new0 (chanview_sort_entry, n);
-	positions = g_new0 (int, n);
-	new_order = g_new0 (gint, n);
-
-	count = 0;
-	for (i = 0; i < n; i++)
-	{
-		new_order[i] = i;
-		if (!gtk_tree_model_iter_nth_child (model, &iter, parent, i))
-			continue;
-		gtk_tree_model_get (model, &iter, COL_CHAN, &ch, -1);
-		if (ch && ch->tag == 0 && ch->userdata)
-		{
-			entries[count].ch = ch;
-			entries[count].old_pos = i;
-			positions[count] = i;
-			count++;
-		}
-	}
-
-	/* stable insertion sort; tab groups stay small so this is cheap */
-	for (i = 1; i < count; i++)
-	{
-		chanview_sort_entry key = entries[i];
-
-		j = i - 1;
-		while (j >= 0 && cv->cb_compare (entries[j].ch->userdata, key.ch->userdata) > 0)
-		{
-			entries[j + 1] = entries[j];
-			j--;
-		}
-		entries[j + 1] = key;
-	}
-
-	for (i = 0; i < count; i++)
-		new_order[positions[i]] = entries[i].old_pos;
-
-	gtk_tree_store_reorder (cv->store, parent, new_order);
-
-	g_free (entries);
-	g_free (positions);
-	g_free (new_order);
-}
-
-/* top-level rows keep their order (insertion never sorts them either);
- * only the children within each family are sorted */
-void
-chanview_resort (chanview *cv)
-{
-	GtkTreeIter iter;
-
-	if (!cv->cb_compare)
-		return;
-
-	if (gtk_tree_model_get_iter_first (GTK_TREE_MODEL (cv->store), &iter))
-	{
-		do
-			chanview_resort_children (cv, &iter);
-		while (gtk_tree_model_iter_next (GTK_TREE_MODEL (cv->store), &iter));
-	}
-}
-
 static void
 chanview_free_ch (chanview *cv, GtkTreeIter *iter)
 {

src/fe-gtk/chanview.h

@@ -34,10 +34,6 @@ void chanview_set_callbacks (chanview *cv,
 	gboolean (*cb_contextmenu) (chanview *, chan *, int tag, void *userdata, GdkEventButton *),
 	int (*cb_compare) (void *a, void *b));
 void chanview_set_impl (chanview *cv, int type);
-void chanview_set_sorted (chanview *cv, gboolean sorted);
-void chanview_set_use_icons (chanview *cv, gboolean use_icons);
-void chanview_set_trunc_len (chanview *cv, int trunc_len);
-void chanview_resort (chanview *cv);
 chan *chanview_add (chanview *cv, char *name, void *family, void *userdata, gboolean allow_closure, int tag, GdkPixbuf *icon);
 int chanview_get_size (chanview *cv);
 GtkWidget *chanview_get_box (chanview *cv);

src/fe-gtk/gtkutil.c

@@ -527,43 +527,6 @@ gtkutil_destroy (GtkWidget * igad, GtkWidget * dgad)
 	gtk_widget_destroy (dgad);
 }
 
-static void
-gtkutil_place_dialog (GtkWidget *dialog)
-{
-	GList *list;
-	GList *item;
-	GtkWindow *anchor = NULL;
-	gint x;
-	gint y;
-	extern GtkWidget *parent_window;
-
-	list = gtk_window_list_toplevels ();
-	for (item = list; item; item = item->next)
-	{
-		if (item->data != dialog && GTK_IS_WINDOW (item->data) &&
-			gtk_widget_get_visible (GTK_WIDGET (item->data)) &&
-			gtk_window_is_active (GTK_WINDOW (item->data)))
-		{
-			anchor = GTK_WINDOW (item->data);
-			break;
-		}
-	}
-	g_list_free (list);
-
-	if (!anchor && parent_window && parent_window != dialog && GTK_IS_WINDOW (parent_window))
-		anchor = GTK_WINDOW (parent_window);
-
-	if (anchor)
-	{
-		gtk_window_get_position (anchor, &x, &y);
-		gtk_window_move (GTK_WINDOW (dialog), x, y);
-	}
-	else
-	{
-		gtk_window_set_position (GTK_WINDOW (dialog), GTK_WIN_POS_MOUSE);
-	}
-}
-
 static void
 gtkutil_get_str_response (GtkDialog *dialog, gint arg1, gpointer entry)
 {
@@ -577,15 +540,15 @@ gtkutil_get_str_response (GtkDialog *dialog, gint arg1, gpointer entry)
 
 	switch (arg1)
 	{
+	case GTK_RESPONSE_REJECT:
+		callback (TRUE, text, user_data);
+		gtk_widget_destroy (GTK_WIDGET (dialog));
+		break;
 	case GTK_RESPONSE_ACCEPT:
 		callback (FALSE, text, user_data);
-		break;
+		gtk_widget_destroy (GTK_WIDGET (dialog));
-	default:
-		callback (TRUE, text, user_data);
 		break;
 	}
-
-	gtk_widget_destroy (GTK_WIDGET (dialog));
 }
 
 static void
@@ -594,8 +557,8 @@ gtkutil_str_enter (GtkWidget *entry, GtkWidget *dialog)
 	gtk_dialog_response (GTK_DIALOG (dialog), GTK_RESPONSE_ACCEPT);
 }
 
-static void
+void
-gtkutil_get_str (char *msg, char *def, void *callback, void *userdata, gboolean visible)
+fe_get_str (char *msg, char *def, void *callback, void *userdata)
 {
 	GtkWidget *dialog;
 	GtkWidget *entry;
@@ -609,11 +572,18 @@ gtkutil_get_str (char *msg, char *def, void *callback, void *userdata, gboolean
 										NULL);
 	theme_manager_attach_window (dialog);
 
-	if (parent_window && GTK_IS_WINDOW (parent_window))
 	gtk_window_set_transient_for (GTK_WINDOW (dialog), GTK_WINDOW (parent_window));
-	gtkutil_place_dialog (dialog);
 	gtk_box_set_homogeneous (GTK_BOX (gtk_dialog_get_content_area (GTK_DIALOG (dialog))), TRUE);
 
+	if (userdata == (void *)1)	/* nick box is usually on the very bottom, make it centered */
+	{
+		gtk_window_set_position (GTK_WINDOW (dialog), GTK_WIN_POS_CENTER);
+	}
+	else
+	{
+		gtk_window_set_position (GTK_WINDOW (dialog), GTK_WIN_POS_MOUSE);
+	}
+
 	hbox = gtkutil_box_new (GTK_ORIENTATION_HORIZONTAL, TRUE, 0);
 
 	g_object_set_data (G_OBJECT (dialog), "cb", callback);
@@ -622,8 +592,7 @@ gtkutil_get_str (char *msg, char *def, void *callback, void *userdata, gboolean
 	entry = gtk_entry_new ();
 	g_signal_connect (G_OBJECT (entry), "activate",
 						 	G_CALLBACK (gtkutil_str_enter), dialog);
-	gtk_entry_set_text (GTK_ENTRY (entry), def ? def : "");
+	gtk_entry_set_text (GTK_ENTRY (entry), def);
-	gtk_entry_set_visibility (GTK_ENTRY (entry), visible);
 	gtk_box_pack_end (GTK_BOX (hbox), entry, 0, 0, 0);
 
 	label = gtk_label_new (msg);
@@ -637,18 +606,6 @@ gtkutil_get_str (char *msg, char *def, void *callback, void *userdata, gboolean
 	gtk_widget_show_all (dialog);
 }
 
-void
-fe_get_str (char *msg, char *def, void *callback, void *userdata)
-{
-	gtkutil_get_str (msg, def, callback, userdata, TRUE);
-}
-
-void
-fe_get_password (char *msg, void *callback, void *userdata)
-{
-	gtkutil_get_str (msg, "", callback, userdata, FALSE);
-}
-
 static void
 gtkutil_get_number_response (GtkDialog *dialog, gint arg1, gpointer spin)
 {
@@ -684,15 +641,15 @@ gtkutil_get_bool_response (GtkDialog *dialog, gint arg1, gpointer spin)
 
 	switch (arg1)
 	{
+	case GTK_RESPONSE_REJECT:
+		callback (0, user_data);
+		gtk_widget_destroy (GTK_WIDGET (dialog));
+		break;
 	case GTK_RESPONSE_ACCEPT:
 		callback (1, user_data);
-		break;
+		gtk_widget_destroy (GTK_WIDGET (dialog));
-	default:
-		callback (0, user_data);
 		break;
 	}
-
-	gtk_widget_destroy (GTK_WIDGET (dialog));
 }
 
 void
@@ -710,10 +667,9 @@ fe_get_int (char *msg, int def, void *callback, void *userdata)
 										_("_OK"), GTK_RESPONSE_ACCEPT,
 										NULL);
 	theme_manager_attach_window (dialog);
-	if (parent_window && GTK_IS_WINDOW (parent_window))
-		gtk_window_set_transient_for (GTK_WINDOW (dialog), GTK_WINDOW (parent_window));
-	gtkutil_place_dialog (dialog);
 	gtk_box_set_homogeneous (GTK_BOX (gtk_dialog_get_content_area (GTK_DIALOG (dialog))), TRUE);
+	gtk_window_set_position (GTK_WINDOW (dialog), GTK_WIN_POS_MOUSE);
+	gtk_window_set_transient_for (GTK_WINDOW (dialog), GTK_WINDOW (parent_window));
 
 	hbox = gtkutil_box_new (GTK_ORIENTATION_HORIZONTAL, TRUE, 0);
 
@@ -751,10 +707,10 @@ fe_get_bool (char *title, char *prompt, void *callback, void *userdata)
 		_("_Yes"), GTK_RESPONSE_ACCEPT,
 		NULL);
 	theme_manager_attach_window (dialog);
-	if (parent_window && GTK_IS_WINDOW (parent_window))
-		gtk_window_set_transient_for (GTK_WINDOW (dialog), GTK_WINDOW (parent_window));
-	gtkutil_place_dialog (dialog);
 	gtk_box_set_homogeneous (GTK_BOX (gtk_dialog_get_content_area (GTK_DIALOG (dialog))), TRUE);
+	gtk_window_set_position (GTK_WINDOW (dialog), GTK_WIN_POS_MOUSE);
+	gtk_window_set_transient_for (GTK_WINDOW (dialog), GTK_WINDOW (parent_window));
+
 
 	g_object_set_data (G_OBJECT (dialog), "cb", callback);
 	g_object_set_data (G_OBJECT (dialog), "ud", userdata);

src/fe-gtk/maingui.c

@@ -569,18 +569,6 @@ mg_create_tab_colors (void)
         away_list = mg_attr_list_create (&gui_palette[THEME_TOKEN_TAB_AWAY], FALSE);
 }
 
-static PangoAttrList *
-mg_attr_list_for_tab_state (int state)
-{
-        if (state & TAB_STATE_NEW_HILIGHT)
-                return nickseen_list;
-        if (state & TAB_STATE_NEW_MSG)
-                return newmsg_list;
-        if (state & TAB_STATE_NEW_DATA)
-                return newdata_list;
-        return plain_list;
-}
-
 static void
 set_window_urgency (GtkWidget *win, gboolean set)
 {
@@ -726,10 +714,7 @@ mg_set_access_icon (session_gui *gui, GdkPixbuf *pix, gboolean away)
 {
         if (gui->op_xpm)
         {
-                /* no change? (the pref check makes sure a live settings
+                if (pix == gtk_image_get_pixbuf (GTK_IMAGE (gui->op_xpm))) /* no change? */
-                   change can still remove the icon) */
-                if (pix == gtk_image_get_pixbuf (GTK_IMAGE (gui->op_xpm)) &&
-                         prefs.hex_gui_input_icon)
                 {
                         mg_set_myself_away (gui, away);
                         return;
@@ -3611,48 +3596,6 @@ mg_create_topicbar (session *sess, GtkWidget *box)
 	mg_create_chanmodebuttons (gui, bbox);
 }
 
-/* re-apply the multi-line topic and inline mode button preferences to an
- * already built topic bar (see mg_create_topicbar () for the layout) */
-
-static void
-mg_update_topicbar_layout (session_gui *gui)
-{
-	GtkWidget *topic = gui->topic_entry;
-	GtkWidget *vbox = gui->topic_bar;
-	GtkWidget *topic_scroll;
-	GtkWidget *hbox;
-	GtkWidget *mode_hbox;
-	GtkWidget *desired_parent;
-	GtkWidget *current_parent;
-	gboolean wrap;
-
-	if (!topic || !GTK_IS_TEXT_VIEW (topic) || !vbox || !gui->topicbutton_box)
-		return;
-
-	wrap = prefs.hex_gui_topicbar_multiline && !prefs.hex_gui_mode_buttons_inline;
-	gtk_text_view_set_wrap_mode (GTK_TEXT_VIEW (topic),
-		wrap ? GTK_WRAP_WORD_CHAR : GTK_WRAP_NONE);
-
-	topic_scroll = gtk_widget_get_parent (topic);
-	hbox = topic_scroll ? gtk_widget_get_parent (topic_scroll) : NULL;
-	mode_hbox = gtk_widget_get_parent (gui->topicbutton_box);
-
-	if (mode_hbox && hbox)
-	{
-		desired_parent = prefs.hex_gui_mode_buttons_inline ? hbox : vbox;
-		current_parent = gtk_widget_get_parent (mode_hbox);
-		if (current_parent && current_parent != desired_parent)
-		{
-			g_object_ref (mode_hbox);
-			gtk_container_remove (GTK_CONTAINER (current_parent), mode_hbox);
-			gtk_box_pack_start (GTK_BOX (desired_parent), mode_hbox, 0, 0, 0);
-			g_object_unref (mode_hbox);
-		}
-	}
-
-	mg_topicbar_update_height (topic);
-}
-
 /* check if a word is clickable */
 
 static int
@@ -4061,12 +4004,8 @@ mg_create_userlist (session_gui *gui, GtkWidget *box)
         gtk_widget_set_margin_end (gui->namelistinfo, 0);
         gtk_widget_set_hexpand (gui->namelistinfo, TRUE);
         gtk_widget_set_halign (gui->namelistinfo, GTK_ALIGN_FILL);
-        /* always pack it so the preference can be toggled without a
-           restart; visibility is controlled explicitly */
-        gtk_widget_set_no_show_all (gui->namelistinfo, TRUE);
-        gtk_box_pack_start (GTK_BOX (vbox), gui->namelistinfo, 0, 0, 0);
         if (prefs.hex_gui_ulist_count)
-                gtk_widget_show (gui->namelistinfo);
+                gtk_box_pack_start (GTK_BOX (vbox), gui->namelistinfo, 0, 0, 0);
 
         gui->user_tree = ulist = userlist_create (vbox);
 
@@ -4965,11 +4904,7 @@ mg_create_menu (session_gui *gui, GtkWidget *table, int away_state)
                                                                                         gui->menu_item);
         gtk_widget_set_hexpand (gui->menu, TRUE);
         gtk_widget_set_vexpand (gui->menu, FALSE);
-        /* Fill the row: with GTK_ALIGN_START the menu bar stops after its
+        gtk_widget_set_halign (gui->menu, GTK_ALIGN_START);
-         * last item and the rest of the row shows bare window background,
-         * which reads as a two-colored menu bar whenever the theme styles
-         * menubar and window differently. */
-        gtk_widget_set_halign (gui->menu, GTK_ALIGN_FILL);
         gtk_widget_set_valign (gui->menu, GTK_ALIGN_FILL);
         gtk_grid_attach (GTK_GRID (table), gui->menu, 0, 0, 3, 1);
 }
@@ -5294,10 +5229,6 @@ mg_apply_setup (void)
                 sess = list->data;
                 gtk_xtext_set_time_stamp (sess->res->buffer, prefs.hex_stamp_text);
                 ((xtext_buffer *)sess->res->buffer)->needs_recalc = TRUE;
-                /* the attribute lists were just recreated, so re-apply the
-                   color matching each tab's activity state */
-                if (sess->res->tab)
-                        chan_set_color (sess->res->tab, mg_attr_list_for_tab_state (sess->tab_state));
                 if (!sess->gui->is_tab || !done_main)
                         mg_place_userlist_and_chanview (sess->gui);
                 if (sess->gui->is_tab)
@@ -5340,12 +5271,14 @@ fe_buttons_update (session *sess)
                 gtk_widget_hide (sess->gui->button_box);
 }
 
-/* tab label for a session that has no joined channel: the channel we are
+void
- * waiting to join in parentheses, or "<none>". tbuf must hold CHANLEN+6. */
+fe_clear_channel (session *sess)
-
-static void
-mg_tab_pending_name (session *sess, char *tbuf)
 {
+        char tbuf[CHANLEN+6];
+        session_gui *gui = sess->gui;
+
+        if (sess->gui->is_tab)
+        {
                 if (sess->waitchannel[0])
                 {
                         if (prefs.hex_gui_tab_trunc > 2 && g_utf8_strlen (sess->waitchannel, -1) > prefs.hex_gui_tab_trunc)
@@ -5362,36 +5295,6 @@ mg_tab_pending_name (session *sess, char *tbuf)
                 }
                 else
                         strcpy (tbuf, _("<none>"));
-}
-
-/* re-apply the tab label truncation preference to a session's tab */
-
-static void
-mg_retrunc_tab (session *sess)
-{
-        char tbuf[CHANLEN+6];
-
-        if (!sess->res->tab)
-                return;
-
-        if (sess->channel[0])
-                chan_rename (sess->res->tab, sess->channel, prefs.hex_gui_tab_trunc);
-        else
-        {
-                mg_tab_pending_name (sess, tbuf);
-                chan_rename (sess->res->tab, tbuf, prefs.hex_gui_tab_trunc);
-        }
-}
-
-void
-fe_clear_channel (session *sess)
-{
-        char tbuf[CHANLEN+6];
-        session_gui *gui = sess->gui;
-
-        if (sess->gui->is_tab)
-        {
-                mg_tab_pending_name (sess, tbuf);
                 chan_rename (sess->res->tab, tbuf, prefs.hex_gui_tab_trunc);
         }
 
@@ -5515,110 +5418,6 @@ fe_set_channel (session *sess)
                 chan_rename (sess->res->tab, sess->channel, prefs.hex_gui_tab_trunc);
 }
 
-static void
-mg_apply_live_prefs_to_gui (session_gui *gui, session *sess, const mg_live_prefs *changes)
-{
-        if (changes->meters)
-                mg_update_meters (gui);
-
-        if (changes->ulist_columns && gui->user_tree)
-                userlist_rebuild_columns (gui->user_tree);
-
-        if (changes->ulist_count && gui->namelistinfo)
-        {
-                if (prefs.hex_gui_ulist_count)
-                        gtk_widget_show (gui->namelistinfo);
-                else
-                        gtk_widget_hide (gui->namelistinfo);
-        }
-
-        if (changes->input_box && gui->nick_box)
-        {
-                if (prefs.hex_gui_input_nick)
-                        gtk_widget_show (gui->nick_box);
-                else
-                        gtk_widget_hide (gui->nick_box);
-        }
-
-        /* refresh the access icon; re-filling the user list rows can
-           clear it as a side effect, so restore it here too */
-        if ((changes->input_box || changes->ulist_rows) && sess && is_session (sess))
-        {
-                if (sess->type == SESS_DIALOG)
-                        mg_set_access_icon (gui, NULL, sess->server->is_away);
-                else
-                        mg_set_access_icon (gui, get_user_icon (sess->server, sess->me), sess->server->is_away);
-        }
-
-        if (changes->topic_bar)
-                mg_update_topicbar_layout (gui);
-
-        if (changes->transparency && gui->window)
-                gtk_widget_set_opacity (gui->window, (prefs.hex_gui_transparency / 255.));
-}
-
-/* apply preference changes that used to require a restart. called by
- * setup.c after the new prefs have been committed. */
-
-void
-mg_apply_live_prefs (const mg_live_prefs *changes)
-{
-        GSList *list;
-        session *sess;
-        int done_main = FALSE;
-
-        if (!changes)
-                return;
-
-        /* the tab bar / channel tree lives in the shared tab window */
-        if (mg_gui && mg_gui->chanview)
-        {
-                if (changes->tab_resort)
-                {
-                        chanview_set_sorted (mg_gui->chanview, prefs.hex_gui_tab_sort);
-                        chanview_resort (mg_gui->chanview);
-                }
-                if (changes->tab_trunc)
-                        chanview_set_trunc_len (mg_gui->chanview, prefs.hex_gui_tab_trunc);
-                if (changes->chanview || changes->tab_resort)
-                {
-                        chanview_set_use_icons (mg_gui->chanview,
-                                                                                        prefs.hex_gui_tab_icons && (pix_tree_channel || pix_tree_dialog ||
-                                                                                                                                                                 pix_tree_server || pix_tree_util));
-                        /* rebuild the tab bar / tree widgets from the model */
-                        chanview_set_impl (mg_gui->chanview, prefs.hex_gui_tab_layout);
-                }
-        }
-
-        list = sess_list;
-        while (list)
-        {
-                sess = list->data;
-
-                if (changes->tab_trunc)
-                        mg_retrunc_tab (sess);
-                if (changes->ulist_sort)
-                        userlist_apply_sort (sess);
-                if (changes->ulist_rows)
-                        userlist_refill (sess);
-
-                if (sess->gui->is_tab)
-                {
-                        /* only apply to the shared tab window once, using the
-                           front-most session for session-dependent bits */
-                        if (!done_main)
-                        {
-                                done_main = TRUE;
-                                mg_apply_live_prefs_to_gui (sess->gui, current_tab, changes);
-                        }
-                }
-                else
-                        mg_apply_live_prefs_to_gui (sess->gui, sess, changes);
-
-                list = list->next;
-        }
-}
-
 void
 fe_set_typing (session *sess, const char *nick, const char *state)
 {

src/fe-gtk/maingui.h

@@ -37,24 +37,6 @@ GtkWidget *mg_create_generic_tab (char *name, char *title, int force_toplevel, i
 void mg_set_title (GtkWidget *button, char *title);
 void mg_set_access_icon (session_gui *gui, GdkPixbuf *pix, gboolean away);
 void mg_apply_setup (void);
-
-/* which parts of the GUI need a live rebuild after a preferences change */
-typedef struct
-{
-	unsigned int chanview:1;			/* rebuild the tab bar / channel tree */
-	unsigned int tab_resort:1;		/* re-sort channel tabs */
-	unsigned int tab_trunc:1;			/* re-truncate tab labels */
-	unsigned int ulist_columns:1;	/* rebuild user list columns */
-	unsigned int ulist_rows:1;		/* re-fill user list rows */
-	unsigned int ulist_sort:1;		/* re-apply user list sort order */
-	unsigned int ulist_count:1;		/* user count label visibility */
-	unsigned int meters:1;				/* rebuild lag/throttle meters */
-	unsigned int input_box:1;			/* nick box visibility and access icon */
-	unsigned int topic_bar:1;			/* topic wrap mode and mode button placement */
-	unsigned int transparency:1;	/* main window opacity */
-} mg_live_prefs;
-
-void mg_apply_live_prefs (const mg_live_prefs *changes);
 void mg_apply_session_font_prefs (session_gui *gui);
 void mg_close_sess (session *);
 void mg_tab_close (session *sess);

src/fe-gtk/menu.c

@@ -138,7 +138,6 @@ struct mymenu
 #define XCMENU_MARKUP 2
 #define XCMENU_MNEMONIC 4
 
-/* execute a userlistbutton/popupmenu command */
 
 static void
 nick_command (session * sess, char *cmd)
@@ -149,7 +148,6 @@ nick_command (session * sess, char *cmd)
 		handle_command (sess, cmd, TRUE);
 }
 
-/* fill in the %a %s %n etc and execute the command */
 
 void
 nick_command_parse (session *sess, char *cmd, char *nick, char *allnick)
@@ -178,7 +176,6 @@ nick_command_parse (session *sess, char *cmd, char *nick, char *allnick)
 		}
 	}
 
-	/* this can't overflow, since popup->cmd is only 256 */
 	len = strlen (cmd) + strlen (nick) + strlen (allnick) + 512;
 	buf = g_malloc (len);
 
@@ -191,7 +188,6 @@ nick_command_parse (session *sess, char *cmd, char *nick, char *allnick)
 	g_free (buf);
 }
 
-/* userlist button has been clicked */
 
 void
 userlist_button_cb (GtkWidget * button, char *cmd)
@@ -208,7 +204,6 @@ userlist_button_cb (GtkWidget * button, char *cmd)
 
 	if (sess->type == SESS_DIALOG)
 	{
-		/* fake a selection */
 		nicks = g_new (char *, 2);
 		nicks[0] = g_strdup (sess->channel);
 		nicks[1] = NULL;
@@ -216,7 +211,6 @@ userlist_button_cb (GtkWidget * button, char *cmd)
 	}
 	else
 	{
-		/* find number of selected rows */
 		nicks = userlist_selection_list (sess->gui->user_tree, &num_sel);
 		if (num_sel < 1)
 		{
@@ -227,7 +221,6 @@ userlist_button_cb (GtkWidget * button, char *cmd)
 		}
 	}
 
-	/* create "allnicks" string */
 	allnicks = g_malloc (((NICKLEN + 1) * num_sel) + 1);
 	*allnicks = 0;
 
@@ -241,7 +234,6 @@ userlist_button_cb (GtkWidget * button, char *cmd)
 		if (!nick)
 			nick = nicks[0];
 
-		/* if not using "%a", execute the command once for each nickname */
 		if (!using_allnicks)
 			nick_command_parse (sess, cmd, nicks[i], "");
 
@@ -265,19 +257,16 @@ userlist_button_cb (GtkWidget * button, char *cmd)
 	g_free (allnicks);
 }
 
-/* a popup-menu-item has been selected */
 
 static void
 popup_menu_cb (GtkWidget * item, char *cmd)
 {
 	char *nick;
 
-	/* the userdata is set in menu_quick_item() */
 	nick = g_object_get_data (G_OBJECT (item), "u");
 
 	if (!nick)	/* userlist popup menu */
 	{
-		/* treat it just like a userlist button */
 		userlist_button_cb (NULL, cmd);
 		return;
 	}
@@ -389,7 +378,6 @@ menu_quick_sub (char *name, GtkWidget *menu, GtkWidget **sub_item_ret, int flags
 	if (!name)
 		return menu;
 
-	/* Code to add a submenu */
 	sub_menu = menu_new ();
 	if (flags & XCMENU_MARKUP)
 	{
@@ -411,7 +399,6 @@ menu_quick_sub (char *name, GtkWidget *menu, GtkWidget **sub_item_ret, int flags
 		*sub_item_ret = sub_item;
 
 	if (flags & XCMENU_DOLIST)
-		/* We create a new element in the list */
 		submenu_list = g_slist_prepend (submenu_list, sub_menu);
 	return sub_menu;
 }
@@ -419,7 +406,6 @@ menu_quick_sub (char *name, GtkWidget *menu, GtkWidget **sub_item_ret, int flags
 static GtkWidget *
 menu_quick_endsub (void)
 {
-	/* Just delete the first element in the linked list pointed to by first */
 	if (submenu_list)
 		submenu_list = g_slist_remove (submenu_list, submenu_list->data);
 
@@ -450,10 +436,7 @@ is_in_path (char *cmd)
 	char **argv;
 	int argc;
 
-	/* special-case these default entries. */
-	/*                  123456789012345678 */
 	if (strncmp (prog, "gnome-terminal -x ", 18) == 0)
-	/* don't check for gnome-terminal, but the thing it's executing! */
 		prog += 18;
 
 	if (g_shell_parse_argv (prog, &argc, &argv, NULL))
@@ -472,7 +455,6 @@ is_in_path (char *cmd)
 	return 0;
 }
 
-/* syntax: "LABEL~ICON~STUFF~ADDED~LATER~" */
 
 static void
 menu_extract_icon (char *name, char **label, char **icon)
@@ -485,7 +467,6 @@ menu_extract_icon (char *name, char **label, char **icon)
 	{
 		if (*p == '~')
 		{
-			/* escape \~ */
 			if (p == name || p[-1] != '\\')
 			{
 				if (!start)
@@ -512,7 +493,6 @@ menu_extract_icon (char *name, char **label, char **icon)
 	}
 }
 
-/* append items to "menu" using the (struct popup*) list provided */
 
 void
 menu_create (GtkWidget *menu, GSList *list, char *target, int check_path)
@@ -539,14 +519,12 @@ menu_create (GtkWidget *menu, GSList *list, char *target, int check_path)
 
 		} else if (!g_ascii_strncasecmp (pop->name, "ENDSUB", 6))
 		{
-			/* empty sub menu due to no programs in PATH? */
 			if (check_path && childcount < 1)
 				gtk_widget_destroy (subitem);
 			subitem = NULL;
 
 			if (tempmenu != menu)
 				tempmenu = menu_quick_endsub ();
-			/* If we get here and tempmenu equals menu that means we havent got any submenus to exit from */
 
 		} else if (!g_ascii_strncasecmp (pop->name, "SEP", 3))
 		{
@@ -556,10 +534,8 @@ menu_create (GtkWidget *menu, GSList *list, char *target, int check_path)
 		{
 			char *icon, *label;
 
-			/* default command in zoitechat.c */
 			if (pop->cmd[0] == 'n' && !strcmp (pop->cmd, "notify -n ASK %s"))
 			{
-				/* don't create this item if already in notify list */
 				if (!target || notify_is_in_list (current_sess->server, target))
 				{
 					list = list->next;
@@ -572,7 +548,6 @@ menu_create (GtkWidget *menu, GSList *list, char *target, int check_path)
 			if (!check_path || pop->cmd[0] != '!')
 			{
 				menu_quick_item (pop->cmd, label, tempmenu, 0, target, icon);
-			/* check if the program is in path, if not, leave it out! */
 			} else if (is_in_path (pop->cmd))
 			{
 				childcount++;
@@ -586,7 +561,6 @@ menu_create (GtkWidget *menu, GSList *list, char *target, int check_path)
 		list = list->next;
 	}
 
-	/* Let's clean up the linked list from mem */
 	while (submenu_list)
 		submenu_list = g_slist_remove (submenu_list, submenu_list->data);
 }
@@ -640,10 +614,8 @@ menu_nickinfo_cb (GtkWidget *menu, session *sess)
 	if (!is_session (sess))
 		return;
 
-	/* issue a /WHOIS */
 	g_snprintf (buf, sizeof (buf), "WHOIS %s %s", str_copy, str_copy);
 	handle_command (sess, buf, FALSE);
-	/* and hide the output */
 	sess->server->skip_next_whois = 1;
 }
 
@@ -653,7 +625,6 @@ copy_to_clipboard_cb (GtkWidget *item, char *url)
 	gtkutil_copy_to_clipboard (item, NULL, url);
 }
 
-/* returns boolean: Some data is missing */
 
 static gboolean
 menu_create_nickinfo_menu (struct User *user, GtkWidget *submenu)
@@ -665,7 +636,6 @@ menu_create_nickinfo_menu (struct User *user, GtkWidget *submenu)
 	gboolean missing = FALSE;
 	GtkWidget *item;
 
-	/* let the translators tweak this if need be */
 	fmt = _("<tt><b>%-11s</b></tt> %s");
 	g_snprintf (unknown, sizeof (unknown), "<i>%s</i>", _("Unknown"));
 
@@ -756,14 +726,11 @@ fe_userlist_update (session *sess, struct User *user)
 	if (!nick_submenu || !str_copy)
 		return;
 
-	/* not the same nick as the menu? */
 	if (sess->server->p_cmp (user->nick, str_copy))
 		return;
 
-	/* get rid of the "show" signal */
 	g_signal_handlers_disconnect_by_func (nick_submenu, menu_nickinfo_cb, sess);
 
-	/* destroy all the old items */
 	items = gtk_container_get_children (GTK_CONTAINER (nick_submenu));
 	iter = items;
 	while (iter)
@@ -774,7 +741,6 @@ fe_userlist_update (session *sess, struct User *user)
 	}
 	g_list_free (items);
 
-	/* and re-create them with new info */
 	needs_refresh = menu_create_nickinfo_menu (user, nick_submenu) ||
 		!user->hostname || !user->realname || !user->servername;
 
@@ -816,7 +782,6 @@ menu_nickmenu (session *sess, GdkEventButton *event, char *nick, int num_sel)
 
 	submenu_list = 0;	/* first time through, might not be 0 */
 
-	/* more than 1 nick selected? */
 	if (num_sel > 1)
 	{
 		g_snprintf (buf, sizeof (buf), _("%d nicks selected."), num_sel);
@@ -861,7 +826,6 @@ menu_nickmenu (session *sess, GdkEventButton *event, char *nick, int num_sel)
 	menu_popup (menu, event, NULL);
 }
 
-/* stuff for the View menu */
 
 static void
 menu_showhide_cb (session *sess)
@@ -1025,7 +989,6 @@ menu_fullscreen_toggle (GtkWidget *wid, gpointer ud)
 #ifdef WIN32
 		if (!prefs.hex_gui_win_state) /* not maximized */
 		{
-			/* other window managers seem to handle this */
 			gtk_window_resize (GTK_WINDOW (parent_window),
 				prefs.hex_gui_win_width, prefs.hex_gui_win_height);
 			gtk_window_move (GTK_WINDOW (parent_window),
@@ -1051,7 +1014,6 @@ open_url_cb (GtkWidget *item, char *url)
 {
 	char buf[512];
 
-	/* pass this to /URL so it can handle irc:// */
 	g_snprintf (buf, sizeof (buf), "URL %s", url);
 	handle_command (current_sess, buf, FALSE);
 }
@@ -1066,7 +1028,6 @@ menu_urlmenu (GdkEventButton *event, char *url)
 	str_copy = g_strdup (url);
 
 	menu = menu_new ();
-	/* more than 51 chars? Chop it */
 	if (g_utf8_strlen (str_copy, -1) >= 52)
 	{
 		tmp = g_strdup (str_copy);
@@ -1081,14 +1042,12 @@ menu_urlmenu (GdkEventButton *event, char *url)
 	}
 	menu_quick_item (0, 0, menu, XCMENU_SHADED, 0, 0);
 
-	/* Two hardcoded entries */
 	if (strncmp (str_copy, "irc://", 6) == 0 ||
 	    strncmp (str_copy, "ircs://",7) == 0)
 		menu_quick_item_with_callback (open_url_cb, _("Connect"), menu, str_copy);
 	else
 		menu_quick_item_with_callback (open_url_cb, _("Open Link in Browser"), menu, str_copy);
 	menu_quick_item_with_callback (copy_to_clipboard_cb, _("Copy Selected Link"), menu, str_copy);
-	/* custom ones from urlhandlers.conf */
 	menu_create (menu, urlhandler_list, str_copy, TRUE);
 	menu_add_plugin_items (menu, "\x4$URL", str_copy);
 	menu_popup (menu, event, NULL);
@@ -1356,7 +1315,6 @@ menu_newserver_tab (GtkWidget * wid, gpointer none)
 	int oldf = prefs.hex_gui_tab_newtofront;
 
 	prefs.hex_gui_tab_chans = 1;
-	/* force focus if setting is "only requested tabs" */
 	if (prefs.hex_gui_tab_newtofront == 2)
 		prefs.hex_gui_tab_newtofront = 1;
 	new_ircwindow (NULL, NULL, SESS_SERVER, 0);
@@ -1740,9 +1698,66 @@ menu_ctcpguiopen (void)
 	editlist_gui_open (NULL, NULL, ctcp_list, buf, "ctcpreply", "ctcpreply.conf", ctcp_help);
 }
 
+/* Returns the index.html of a locally installed copy of the documentation,
+ * or NULL if none is available. Callers own the returned string. */
+static char *
+menu_docs_local_index (void)
+{
+	const char *env_docdir;
+	char *path;
+
+	env_docdir = g_getenv ("ZOITECHAT_DOCDIR");
+	if (env_docdir && env_docdir[0])
+	{
+		path = g_build_filename (env_docdir, "index.html", NULL);
+		if (g_file_test (path, G_FILE_TEST_IS_REGULAR))
+			return path;
+		g_free (path);
+	}
+
+	/* Docs the user placed in the config dir; works on every platform,
+	 * including portable installs and Flatpak (host-readable, unlike /app) */
+	path = g_build_filename (get_xdir (), "offline-docs", "index.html", NULL);
+	if (g_file_test (path, G_FILE_TEST_IS_REGULAR))
+		return path;
+	g_free (path);
+
+#ifdef WIN32
+	/* The installer's "Offline Documentation" component; ZOITECHATDOCDIR is
+	 * relative to the install dir on Windows and must not be resolved
+	 * against the CWD, which depends on how the app was launched */
+	if (!g_path_is_absolute (ZOITECHATDOCDIR))
+	{
+		char *install_dir = g_win32_get_package_installation_directory_of_module (NULL);
+
+		if (!install_dir)
+			return NULL;
+		path = g_build_filename (install_dir, ZOITECHATDOCDIR, "index.html", NULL);
+		g_free (install_dir);
+	}
+	else
+		path = g_build_filename (ZOITECHATDOCDIR, "index.html", NULL);
+#else
+	path = g_build_filename (ZOITECHATDOCDIR, "index.html", NULL);
+#endif
+	if (g_file_test (path, G_FILE_TEST_IS_REGULAR))
+		return path;
+	g_free (path);
+
+	return NULL;
+}
+
 static void
 menu_docs (GtkWidget *wid, gpointer none)
 {
+	char *local_index = menu_docs_local_index ();
+
+	if (local_index)
+	{
+		fe_open_url (local_index);
+		g_free (local_index);
+	}
+	else
 		fe_open_url ("https://docs.zoitechat.org/en/latest/");
 }
 
@@ -2252,7 +2267,6 @@ menu_canacaccel (GtkWidget *widget, guint signal_id, gpointer user_data)
 	return gtk_widget_is_sensitive (widget);
 }
 
-/* === STUFF FOR /MENU === */
 
 static GtkMenuItem *
 menu_find_item (GtkWidget *menu, char *name)
@@ -2318,7 +2332,6 @@ menu_find_path (GtkWidget *menu, char *path)
 	char name[128];
 	int len;
 
-	/* grab the next part of the path */
 	s = strchr (path, '/');
 	len = s - path;
 	if (!s)
@@ -2372,7 +2385,6 @@ menu_foreach_gui (menu_entry *me, void (*callback) (GtkWidget *, menu_entry *, c
 			list = list->next;
 			continue;
 		}
-		/* do it only once for tab sessions, since they share a GUI */
 		if (!sess->gui->is_tab || !tabdone)
 		{
 			callback (sess->gui->menu, me, NULL);
@@ -2392,13 +2404,11 @@ menu_update_cb (GtkWidget *menu, menu_entry *me, char *target)
 	if (item)
 	{
 		gtk_widget_set_sensitive (item, me->enable);
-		/* must do it without triggering the callback */
 		if (GTK_IS_CHECK_MENU_ITEM (item))
 			gtk_check_menu_item_set_active (GTK_CHECK_MENU_ITEM (item), me->state);
 	}
 }
 
-/* radio state changed via mouse click */
 static void
 menu_radio_cb (GtkCheckMenuItem *item, menu_entry *me)
 {
@@ -2406,15 +2416,12 @@ menu_radio_cb (GtkCheckMenuItem *item, menu_entry *me)
 	if (gtk_check_menu_item_get_active (item))
 		me->state = 1;
 
-	/* update the state, incase this was changed via right-click. */
-	/* This will update all other windows and menu bars */
 	menu_foreach_gui (me, menu_update_cb);
 
 	if (me->state && me->cmd)
 		handle_command (current_sess, me->cmd, FALSE);
 }
 
-/* toggle state changed via mouse click */
 static void
 menu_toggle_cb (GtkCheckMenuItem *item, menu_entry *me)
 {
@@ -2422,8 +2429,6 @@ menu_toggle_cb (GtkCheckMenuItem *item, menu_entry *me)
 	if (gtk_check_menu_item_get_active (item))
 		me->state = 1;
 
-	/* update the state, incase this was changed via right-click. */
-	/* This will update all other windows and menu bars */
 	menu_foreach_gui (me, menu_update_cb);
 
 	if (me->state)
@@ -2594,7 +2599,6 @@ fe_menu_add (menu_entry *me)
 	if (!pango_parse_markup (me->label, -1, 0, NULL, &text, NULL, NULL))
 		return NULL;
 
-	/* return the label with markup stripped */
 	return text;
 }
 
@@ -2610,7 +2614,6 @@ fe_menu_update (menu_entry *me)
 	menu_foreach_gui (me, menu_update_cb);
 }
 
-/* used to add custom menus to the right-click menu */
 
 static void
 menu_add_plugin_mainmenu_items (GtkWidget *menu)
@@ -2644,7 +2647,6 @@ menu_add_plugin_items (GtkWidget *menu, char *root, char *target)
 	}
 }
 
-/* === END STUFF FOR /MENU === */
 
 GtkWidget *
 menu_create_main (void *accel_group, int bar, int away, int toplevel,
@@ -2677,13 +2679,11 @@ menu_create_main (void *accel_group, int bar, int away, int toplevel,
 	else
 		menu_bar = menu_new ();
 
-	/* /MENU needs to know this later */
 	g_object_set_data (G_OBJECT (menu_bar), "accel", accel_group);
 
 	g_signal_connect (G_OBJECT (menu_bar), "can-activate-accel",
 							G_CALLBACK (menu_canacaccel), 0);
 
-	/* set the initial state of toggles */
 	mymenu[MENUBAR_OFFSET].state = !prefs.hex_gui_hide_menu;
 	mymenu[MENUBAR_OFFSET+1].state = prefs.hex_gui_topicbar;
 	mymenu[MENUBAR_OFFSET+2].state = !prefs.hex_gui_ulist_hide;
@@ -2722,7 +2722,6 @@ menu_create_main (void *accel_group, int bar, int away, int toplevel,
 		mymenu[METRE_OFFSET+3].state = 1;
 	}
 
-	/* change Close binding to ctrl-shift-w when using emacs keys */
 	settings = gtk_widget_get_settings (menu_bar);
 	if (settings)
 	{
@@ -2738,7 +2737,6 @@ menu_create_main (void *accel_group, int bar, int away, int toplevel,
 		}
 	}
 
-	/* Away binding to ctrl-alt-a if the _Help menu conflicts (FR/PT/IT) */
 	{
 		char *help = _("_Help");
 		char *under = strchr (help, '_');
@@ -2775,7 +2773,6 @@ menu_create_main (void *accel_group, int bar, int away, int toplevel,
 			if (mymenu[i].id == MENU_ID_USERMENU)
 				usermenu = menu;
 			menu_item = gtk_menu_item_new_with_mnemonic (_(mymenu[i].text));
-			/* record the English name for /menu */
 			g_object_set_data (G_OBJECT (menu_item), "name", mymenu[i].text);
 #ifdef HAVE_GTK_MAC /* Added to app menu, see below */
 			if (!bar || mymenu[i].id != MENU_ID_ZOITECHAT)		
@@ -2827,7 +2824,6 @@ normalitem:
 togitem:
 			g_object_set_data (G_OBJECT (item), "zc-key-action", (gpointer) menu_get_key_action_name (i));
 			menu_add_keybinding_accel (item, accel_group, menu_get_key_action_name (i));
-			/* must avoid callback for Radio buttons */
 			gtk_check_menu_item_set_active (GTK_CHECK_MENU_ITEM (item), mymenu[i].state);
 			/*gtk_check_menu_item_set_active (GTK_CHECK_MENU_ITEM (item),
 													 mymenu[i].state);*/
@@ -2867,7 +2863,6 @@ togitem:
 			group = NULL;
 			submenu = menu_new ();
 			item = create_icon_menu (_(mymenu[i].text), mymenu[i].image, TRUE);
-			/* record the English name for /menu */
 			g_object_set_data (G_OBJECT (item), "name", mymenu[i].text);
 			gtk_menu_item_set_submenu (GTK_MENU_ITEM (item), submenu);
 			gtk_menu_shell_append (GTK_MENU_SHELL (menu), item);
@@ -2889,13 +2884,10 @@ togitem:
 			submenu = NULL;
 		}
 
-		/* record this GtkWidget * so it's state might be changed later */
 		if (mymenu[i].id != 0 && menu_widgets)
-			/* this ends up in sess->gui->menu_item[MENU_ID_XXX] */
 			menu_widgets[mymenu[i].id] = item;
 
 #ifdef HAVE_GTK_MAC
-		/* We want ZoiteChat to be the app menu, not including Quit or ZoiteChat itself */
 		if (bar && item && i <= CLOSE_OFFSET + 1 && mymenu[i].id != MENU_ID_ZOITECHAT)
 		{
 			if (!submenu || mymenu[i].type == M_MENUSUB)

src/fe-gtk/meson.build

@@ -227,20 +227,6 @@ test('Theme Access Routing Tests', theme_access_tests,
   timeout: 120,
 )
 
-theme_gtk3_settings_tests = executable('theme_gtk3_settings_tests',
-  [
-    'theme/tests/test-theme-gtk3-settings.c',
-    'theme/theme-gtk3.c',
-  ],
-  include_directories: [config_h_include],
-  dependencies: [gtk_dep],
-)
-
-test('Theme GTK3 Settings Tests', theme_gtk3_settings_tests,
-  protocol: 'tap',
-  timeout: 120,
-)
-
 theme_preferences_gtk3_populate_tests = executable('theme_preferences_gtk3_populate_tests',
   'theme/tests/test-theme-preferences-gtk3-populate.c',
   include_directories: [config_h_include],

src/fe-gtk/plugin-tray.c

@@ -1539,11 +1539,6 @@ tray_cleanup (void)
 void
 tray_apply_setup (void)
 {
-	/* Theme/setup events can be dispatched before the tray plugin has
-	 * been registered; there is nothing to (de)activate yet. */
-	if (!ph)
-		return;
-
 	if (tray_backend_active)
 	{
 		if (!prefs.hex_gui_tray)

src/fe-gtk/servlistgui.c

@@ -88,7 +88,6 @@ static GtkWidget *edit_entry_real;
 static GtkWidget *edit_entry_pass;
 static GtkWidget *edit_check_show_pass;
 static GtkWidget *edit_check_use_keyring;
-static GtkWidget *edit_check_ask_pass;
 static GtkWidget *edit_button_encrypt_pass;
 static GtkWidget *edit_button_import_pass;
 static int edit_pass_changed;
@@ -123,11 +122,7 @@ servlist_update_password_tools (ircnet *net)
 		return;
 
 	use_keyring = net && (net->flags & FLAG_USE_KEYRING);
-	if ((net && (net->flags & FLAG_PROMPT_PASSWORD)) ||
+	has_local = net && net->pass && *net->pass && !use_keyring && !edit_pass_changed;
-		(edit_check_ask_pass && gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (edit_check_ask_pass))))
-		use_keyring = FALSE;
-	has_local = net && net->pass && *net->pass && !use_keyring && !edit_pass_changed &&
-		(!edit_check_ask_pass || !gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (edit_check_ask_pass)));
 	gtk_widget_set_sensitive (edit_button_encrypt_pass, has_local && !servlist_password_is_encrypted (net->pass));
 	gtk_widget_set_sensitive (edit_button_import_pass, has_local);
 }
@@ -140,17 +135,6 @@ servlist_entry_set_text_silent (GtkWidget *entry, const char *text)
 	g_signal_handlers_unblock_by_func (G_OBJECT (entry), G_CALLBACK (servlist_password_changed_cb), NULL);
 }
 
-static void
-servlist_toggle_ask_pass_cb (GtkToggleButton *toggle, gpointer userdata)
-{
-	gboolean active = gtk_toggle_button_get_active (toggle);
-
-	gtk_widget_set_sensitive (edit_check_use_keyring, !active);
-	gtk_widget_set_sensitive (edit_entry_pass, !active && (!selected_net || selected_net->logintype != LOGIN_SASLEXTERNAL));
-	gtk_widget_set_sensitive (edit_check_show_pass, !active);
-	servlist_update_password_tools (selected_net);
-}
-
 static char *
 servlist_display_password (ircnet *net)
 {
@@ -1277,7 +1261,6 @@ static void
 servlist_edit_update (ircnet *net)
 {
 	gboolean use_keyring;
-	gboolean ask_pass;
 	gboolean keyring_changed;
 	char *password = NULL;
 	servlist_update_from_entry (&net->nick, edit_entry_nick);
@@ -1286,23 +1269,8 @@ servlist_edit_update (ircnet *net)
 	servlist_update_from_entry (&net->real, edit_entry_real);
 	if (net && net->name)
 	{
-		ask_pass = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (edit_check_ask_pass));
+		use_keyring = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (edit_check_use_keyring));
-		use_keyring = !ask_pass && gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (edit_check_use_keyring));
 		keyring_changed = !!(net->flags & FLAG_USE_KEYRING) != !!use_keyring;
-		if (ask_pass)
-		{
-			secretstore_delete_network_password (net->name);
-			if (net->pass)
-			{
-				memset (net->pass, 0, strlen (net->pass));
-				g_free (net->pass);
-				net->pass = NULL;
-			}
-			net->flags &= ~FLAG_USE_KEYRING;
-			net->flags |= FLAG_PROMPT_PASSWORD;
-			return;
-		}
-		net->flags &= ~FLAG_PROMPT_PASSWORD;
 		if (!edit_pass_changed && !keyring_changed)
 			return;
 		if (edit_pass_changed)
@@ -1370,7 +1338,6 @@ servlist_edit_close_cb (GtkWidget *button, gpointer userdata)
 	edit_win = NULL;
 	edit_entry_pass = NULL;
 	edit_check_show_pass = NULL;
-	edit_check_ask_pass = NULL;
 	edit_button_encrypt_pass = NULL;
 	edit_button_import_pass = NULL;
 }
@@ -2598,7 +2565,7 @@ servlist_open_edit (GtkWidget *parent, ircnet *net)
 
 
 	/* Checkboxes and entries */
-	table3 = gtkutil_grid_new (19, 2, FALSE);
+	table3 = gtkutil_grid_new (17, 2, FALSE);
 	gtk_box_pack_start (GTK_BOX (vbox5), table3, FALSE, FALSE, 0);
 	gtk_grid_set_row_spacing (GTK_GRID (table3), 2);
 	gtk_grid_set_column_spacing (GTK_GRID (table3), 8);
@@ -2617,43 +2584,34 @@ servlist_open_edit (GtkWidget *parent, ircnet *net)
 #endif
 	servlist_create_check (1, net->flags & FLAG_USE_GLOBAL, table3, 5, 0, _("Use global user information"));
 
-	edit_check_ask_pass = gtk_check_button_new_with_mnemonic (_("Ask for password on connect"));
-	gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (edit_check_ask_pass), net->flags & FLAG_PROMPT_PASSWORD);
-	servlist_table_attach (table3, edit_check_ask_pass, 0, 2, 6, 7,
-					   FALSE, FALSE,
-					   SERVLIST_ALIGN_START, SERVLIST_ALIGN_CENTER,
-					   SERVLIST_X_PADDING, SERVLIST_Y_PADDING);
-	g_signal_connect (G_OBJECT (edit_check_ask_pass), "toggled",
-				  G_CALLBACK (servlist_toggle_ask_pass_cb), NULL);
-
 	edit_check_use_keyring = gtk_check_button_new_with_mnemonic (_("Use system keyring"));
 	gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (edit_check_use_keyring), net->flags & FLAG_USE_KEYRING);
-	servlist_table_attach (table3, edit_check_use_keyring, 0, 2, 7, 8,
+	servlist_table_attach (table3, edit_check_use_keyring, 0, 2, 6, 7,
 					   FALSE, FALSE,
 					   SERVLIST_ALIGN_START, SERVLIST_ALIGN_CENTER,
 					   SERVLIST_X_PADDING, SERVLIST_Y_PADDING);
 	g_signal_connect (G_OBJECT (edit_check_use_keyring), "toggled",
 				  G_CALLBACK (servlist_toggle_keyring_cb), NULL);
 
-	edit_entry_nick = servlist_create_entry (table3, _("_Nick name:"), 8, net->nick, &edit_label_nick, 0);
+	edit_entry_nick = servlist_create_entry (table3, _("_Nick name:"), 7, net->nick, &edit_label_nick, 0);
-	edit_entry_nick2 = servlist_create_entry (table3, _("Second choice:"), 9, net->nick2, &edit_label_nick2, 0);
+	edit_entry_nick2 = servlist_create_entry (table3, _("Second choice:"), 8, net->nick2, &edit_label_nick2, 0);
-	edit_entry_real = servlist_create_entry (table3, _("Rea_l name:"), 10, net->real, &edit_label_real, 0);
+	edit_entry_real = servlist_create_entry (table3, _("Rea_l name:"), 9, net->real, &edit_label_real, 0);
-	edit_entry_user = servlist_create_entry (table3, _("_User name:"), 11, net->user, &edit_label_user, 0);
+	edit_entry_user = servlist_create_entry (table3, _("_User name:"), 10, net->user, &edit_label_user, 0);
 
 	label_logintype = gtk_label_new (_("Login method:"));
-	servlist_table_attach (table3, label_logintype, 0, 1, 12, 13,
+	servlist_table_attach (table3, label_logintype, 0, 1, 11, 12,
 						   FALSE, FALSE,
 						   SERVLIST_ALIGN_START, SERVLIST_ALIGN_CENTER,
 						   SERVLIST_X_PADDING, SERVLIST_Y_PADDING);
 	gtk_widget_set_halign (label_logintype, GTK_ALIGN_START);
 	gtk_widget_set_valign (label_logintype, GTK_ALIGN_CENTER);
 	combobox_logintypes = servlist_create_logintypecombo (notebook);
-	servlist_table_attach (table3, combobox_logintypes, 1, 2, 12, 13,
+	servlist_table_attach (table3, combobox_logintypes, 1, 2, 11, 12,
 						   FALSE, FALSE,
 						   SERVLIST_ALIGN_FILL, SERVLIST_ALIGN_FILL,
 						   4, 2);
 
-	edit_entry_pass = servlist_create_entry (table3, _("Password:"), 13, NULL, 0, _("Password used for login. If in doubt, leave blank."));
+	edit_entry_pass = servlist_create_entry (table3, _("Password:"), 12, NULL, 0, _("Password used for login. If in doubt, leave blank."));
 	if (edit_loaded_password)
 	{
 		memset (edit_loaded_password, 0, strlen (edit_loaded_password));
@@ -2686,7 +2644,7 @@ servlist_open_edit (GtkWidget *parent, ircnet *net)
 	if (selected_net && selected_net->logintype == LOGIN_SASLEXTERNAL)
 		gtk_widget_set_sensitive (edit_entry_pass, FALSE);
 	edit_check_show_pass = gtk_check_button_new_with_mnemonic (_("Show password"));
-	servlist_table_attach (table3, edit_check_show_pass, 0, 2, 14, 15,
+	servlist_table_attach (table3, edit_check_show_pass, 0, 2, 13, 14,
 						   FALSE, FALSE,
 						   SERVLIST_ALIGN_START, SERVLIST_ALIGN_CENTER,
 						   4, 2);
@@ -2694,14 +2652,14 @@ servlist_open_edit (GtkWidget *parent, ircnet *net)
 					  G_CALLBACK (servlist_toggle_show_password_cb), edit_entry_pass);
 
 	edit_button_encrypt_pass = gtk_button_new_with_mnemonic (_("Encrypt saved password"));
-	servlist_table_attach (table3, edit_button_encrypt_pass, 0, 1, 15, 16,
+	servlist_table_attach (table3, edit_button_encrypt_pass, 0, 1, 14, 15,
 						   FALSE, FALSE,
 						   SERVLIST_ALIGN_START, SERVLIST_ALIGN_CENTER,
 						   SERVLIST_X_PADDING, SERVLIST_Y_PADDING);
 	g_signal_connect (G_OBJECT (edit_button_encrypt_pass), "clicked",
 					  G_CALLBACK (servlist_encrypt_password_cb), net);
 	edit_button_import_pass = gtk_button_new_with_mnemonic (_("Move password to keyring"));
-	servlist_table_attach (table3, edit_button_import_pass, 1, 2, 15, 16,
+	servlist_table_attach (table3, edit_button_import_pass, 1, 2, 14, 15,
 						   FALSE, FALSE,
 						   SERVLIST_ALIGN_START, SERVLIST_ALIGN_CENTER,
 						   4, 2);
@@ -2709,14 +2667,14 @@ servlist_open_edit (GtkWidget *parent, ircnet *net)
 					  G_CALLBACK (servlist_import_password_cb), net);
 
 	label34 = gtk_label_new (_("Character set:"));
-	servlist_table_attach (table3, label34, 0, 1, 16, 17,
+	servlist_table_attach (table3, label34, 0, 1, 15, 16,
 						   FALSE, FALSE,
 						   SERVLIST_ALIGN_START, SERVLIST_ALIGN_CENTER,
 						   SERVLIST_X_PADDING, SERVLIST_Y_PADDING);
 	gtk_widget_set_halign (label34, GTK_ALIGN_START);
 	gtk_widget_set_valign (label34, GTK_ALIGN_CENTER);
 	comboboxentry_charset = servlist_create_charsetcombo ();
-	servlist_table_attach (table3, comboboxentry_charset, 1, 2, 16, 17,
+	servlist_table_attach (table3, comboboxentry_charset, 1, 2, 15, 16,
 						   FALSE, FALSE,
 						   SERVLIST_ALIGN_FILL, SERVLIST_ALIGN_FILL,
 						   4, 2);
@@ -2742,7 +2700,7 @@ servlist_open_edit (GtkWidget *parent, ircnet *net)
 							G_CALLBACK (servlist_delete_client_cert_cb), net);
 	gtk_box_pack_start (GTK_BOX (hbox_cert_buttons), edit_button_cert_delete, FALSE, FALSE, 0);
 
-	servlist_table_attach (table3, hbox_cert_buttons, 0, 2, 17, 18,
+	servlist_table_attach (table3, hbox_cert_buttons, 0, 2, 16, 17,
 						   FALSE, FALSE,
 						   SERVLIST_ALIGN_START, SERVLIST_ALIGN_CENTER,
 						   SERVLIST_X_PADDING, SERVLIST_Y_PADDING);

src/fe-gtk/setup.c

@@ -1045,30 +1045,28 @@ setup_create_spin (GtkWidget *table, int row, const setting *set)
         return wid;
 }
 
-static guint setup_trans_apply_tag = 0;
-/* the window opacity the dialog opened with, so cancelling can undo the
- * live preview done by the slider */
-static int setup_transparency_snapshot;
-
 static gint
-setup_apply_trans (gpointer userdata)
+setup_apply_trans (int *tag)
 {
         prefs.hex_gui_transparency = setup_prefs.hex_gui_transparency;
-        if (current_sess)
         gtk_widget_set_opacity (current_sess->gui->window,
                                                         (prefs.hex_gui_transparency / 255.));
 
-        setup_trans_apply_tag = 0;
+        *tag = 0;
         return 0;
 }
 
 static void
 setup_hscale_cb (GtkRange *wid, const setting *set)
 {
+        static int tag = 0;
+
         setup_set_int (&setup_prefs, set, (int) gtk_range_get_value (wid));
 
-        if (setup_trans_apply_tag == 0)
+        if (tag == 0)
-                setup_trans_apply_tag = g_idle_add ((GSourceFunc) setup_apply_trans, NULL);
+        {
+                tag = g_idle_add ((GSourceFunc) setup_apply_trans, &tag);
+        }
 }
 
 static void
@@ -1093,8 +1091,7 @@ setup_create_hscale (GtkWidget *table, int row, const setting *set)
 
 #ifndef WIN32 /* Windows always supports this */
         /* Only used for transparency currently */
-        if (current_sess &&
+        if (!gdk_screen_is_composited (gtk_widget_get_screen (current_sess->gui->window)))
-                 !gdk_screen_is_composited (gtk_widget_get_screen (current_sess->gui->window)))
                 gtk_widget_set_sensitive (wid, FALSE);
 #endif
 }
@@ -2296,9 +2293,7 @@ setup_apply (struct zoitechatprefs *pr)
 	ThemeChangedEvent event;
 	struct zoitechatprefs old_prefs = prefs;
 	int old_dark_mode = prefs.hex_gui_dark_mode;
-	mg_live_prefs live_changes;
 
-	memset (&live_changes, 0, sizeof (live_changes));
 
 #define DIFF(a) (pr->a != prefs.a)
 
@@ -2306,49 +2301,40 @@ setup_apply (struct zoitechatprefs *pr)
         if (DIFF (hex_gui_lang))
                 noapply = TRUE;
 #endif
-        /* server tabs are only created when a connection is opened, so
-           existing connections keep their current tabs until a restart;
-           the channel tree indentation is still re-applied below */
-        if (DIFF (hex_gui_tab_server))
-        {
-                noapply = TRUE;
-                live_changes.chanview = TRUE;
-        }
-
         if (DIFF (hex_gui_compact))
-        {
+                noapply = TRUE;
-                live_changes.chanview = TRUE;
+        if (DIFF (hex_gui_input_icon))
-                live_changes.ulist_columns = TRUE;
+                noapply = TRUE;
-        }
+        if (DIFF (hex_gui_input_nick))
-        if (DIFF (hex_gui_input_icon) || DIFF (hex_gui_input_nick))
+                noapply = TRUE;
-                live_changes.input_box = TRUE;
+        if (DIFF (hex_gui_lagometer))
-        if (DIFF (hex_gui_lagometer) || DIFF (hex_gui_throttlemeter))
+                noapply = TRUE;
-                live_changes.meters = TRUE;
+        if (DIFF (hex_gui_mode_buttons_inline))
-        if (DIFF (hex_gui_mode_buttons_inline) || DIFF (hex_gui_topicbar_multiline))
+                noapply = TRUE;
-                live_changes.topic_bar = TRUE;
+        if (DIFF (hex_gui_tab_icons))
-        if (DIFF (hex_gui_tab_icons) || DIFF (hex_gui_tab_closebuttons) ||
+                noapply = TRUE;
-                 DIFF (hex_gui_tab_small))
+        if (DIFF (hex_gui_tab_closebuttons))
-                live_changes.chanview = TRUE;
+                noapply = TRUE;
+        if (DIFF (hex_gui_tab_server))
+                noapply = TRUE;
+        if (DIFF (hex_gui_tab_small))
+                noapply = TRUE;
         if (DIFF (hex_gui_tab_sort))
-                live_changes.tab_resort = TRUE;
+                noapply = TRUE;
         if (DIFF (hex_gui_tab_trunc))
-                live_changes.tab_trunc = TRUE;
+                noapply = TRUE;
+        if (DIFF (hex_gui_throttlemeter))
+                noapply = TRUE;
+        if (DIFF (hex_gui_topicbar_multiline))
+                noapply = TRUE;
         if (DIFF (hex_gui_ulist_count))
-                live_changes.ulist_count = TRUE;
+                noapply = TRUE;
         if (DIFF (hex_gui_ulist_icons))
-                live_changes.ulist_rows = TRUE;
+                noapply = TRUE;
         if (DIFF (hex_gui_ulist_show_hosts))
-        {
+                noapply = TRUE;
-                live_changes.ulist_columns = TRUE;
-                live_changes.ulist_rows = TRUE; /* hostnames may be stale */
-        }
         if (DIFF (hex_gui_ulist_sort))
-                live_changes.ulist_sort = TRUE;
+                noapply = TRUE;
-        /* compare against the pre-dialog value: the slider previews the
-           opacity live on the current window only, so OK still needs to
-           apply it to the other windows */
-        if (pr->hex_gui_transparency != setup_transparency_snapshot)
-                live_changes.transparency = TRUE;
 
         if ((pr->hex_gui_tab_pos == 5 || pr->hex_gui_tab_pos == 6) &&
                  pr->hex_gui_tab_layout == 2 && pr->hex_gui_tab_pos != prefs.hex_gui_tab_pos)
@@ -2392,8 +2378,6 @@ setup_apply (struct zoitechatprefs *pr)
 
 	theme_manager_dispatch_setup_apply (&event);
 
-	mg_apply_live_prefs (&live_changes);
-
         if (noapply)
                 fe_message (_("Some settings were changed that require a"
                                                 " restart to take full effect."), FE_MSG_WARN);
@@ -2426,9 +2410,6 @@ setup_ok_cb (GtkWidget *but, GtkWidget *win)
         if (save_result.success)
         {
                 theme_preferences_stage_commit ();
-                /* the previewed values were accepted, so the close handler
-                   must not revert them */
-                setup_transparency_snapshot = prefs.hex_gui_transparency;
                 gtk_widget_destroy (win);
                 return;
         }
@@ -2491,24 +2472,6 @@ setup_close_cb (GtkWidget *win, GtkWidget **swin)
 {
         *swin = NULL;
 
-        /* drop any pending transparency preview and undo the previewed
-           opacity when the dialog is cancelled; after OK the snapshot
-           was updated so this is a no-op */
-        if (setup_trans_apply_tag)
-        {
-                g_source_remove (setup_trans_apply_tag);
-                setup_trans_apply_tag = 0;
-        }
-        if (prefs.hex_gui_transparency != setup_transparency_snapshot)
-        {
-                mg_live_prefs revert_changes;
-
-                prefs.hex_gui_transparency = setup_transparency_snapshot;
-                memset (&revert_changes, 0, sizeof (revert_changes));
-                revert_changes.transparency = TRUE;
-                mg_apply_live_prefs (&revert_changes);
-        }
-
         theme_preferences_stage_discard ();
 
         if (font_dialog)
@@ -2530,7 +2493,6 @@ setup_open (void)
         memcpy (&setup_prefs, &prefs, sizeof (prefs));
 
         color_change = FALSE;
-        setup_transparency_snapshot = prefs.hex_gui_transparency;
         theme_preferences_stage_begin ();
         setup_window = setup_window_open ();
 

src/fe-gtk/theme/tests/test-theme-access-routing.c

@@ -23,7 +23,6 @@
 #include <math.h>
 
 #include "../theme-access.h"
-#include "../theme-css.h"
 #include "../theme-manager.h"
 #include "../theme-runtime.h"
 #include "../../xtext-color.h"
@@ -55,13 +54,6 @@ setup_apply_real (const ThemeChangedEvent *event)
 	(void) event;
 }
 
-void
-gtkutil_append_font_css (GString *css, const PangoFontDescription *font_desc)
-{
-	(void) css;
-	(void) font_desc;
-}
-
 gboolean
 fe_dark_mode_is_enabled_for (unsigned int mode)
 {
@@ -181,14 +173,6 @@ theme_gtk3_is_active (void)
 	return stub_gtk3_active;
 }
 
-gboolean
-theme_gtk3_lookup_theme_color (const char *name, GdkRGBA *out_color)
-{
-	(void) name;
-	(void) out_color;
-	return FALSE;
-}
-
 static gboolean
 rgba_equal (const GdkRGBA *a, const GdkRGBA *b)
 {
@@ -352,79 +336,6 @@ test_access_xtext_palette_widget_mapping_when_gtk3_active (void)
 	g_object_unref (provider);
 }
 
-static void
-test_access_palette_css_does_not_feed_back_into_sampling (void)
-{
-	GtkWidget *window;
-	GtkWidget *label;
-	GtkCssProvider *theme1;
-	GtkCssProvider *theme2;
-	GdkScreen *screen;
-	XTextColor palette[2] = { 0 };
-	GdkRGBA old_bg;
-	GdkRGBA old_fg;
-	GdkRGBA expected;
-
-	if (!gtk_available)
-	{
-		g_test_message ("GTK display not available");
-		return;
-	}
-
-	reset_stubs ();
-	stub_gtk3_active = TRUE;
-	screen = gdk_screen_get_default ();
-	window = gtk_window_new (GTK_WINDOW_TOPLEVEL);
-	label = gtk_label_new ("mapped");
-	gtk_container_add (GTK_CONTAINER (window), label);
-	gtk_widget_realize (window);
-
-	/* Old GTK3 theme active at screen level. */
-	theme1 = gtk_css_provider_new ();
-	gtk_css_provider_load_from_data (theme1,
-		"label { color: #112233; background-color: #445566; }"
-		"label:selected { color: #778899; background-color: #aabbcc; }",
-		-1, NULL);
-	gtk_style_context_add_provider_for_screen (screen,
-		GTK_STYLE_PROVIDER (theme1), GTK_STYLE_PROVIDER_PRIORITY_USER);
-
-	/* The app paints the sampled theme colors onto the widget, like the
-	 * user list / channel tree / input box palette provider does. */
-	g_assert_true (gdk_rgba_parse (&old_bg, "#445566"));
-	g_assert_true (gdk_rgba_parse (&old_fg, "#112233"));
-	theme_css_apply_palette_widget (label, &old_bg, &old_fg, NULL);
-
-	/* The user switches to a new GTK3 theme. */
-	gtk_style_context_remove_provider_for_screen (screen, GTK_STYLE_PROVIDER (theme1));
-	theme2 = gtk_css_provider_new ();
-	gtk_css_provider_load_from_data (theme2,
-		"label { color: #00ff00; background-color: #ff0000; }"
-		"label:selected { color: #0000ff; background-color: #ffff00; }",
-		-1, NULL);
-	gtk_style_context_add_provider_for_screen (screen,
-		GTK_STYLE_PROVIDER (theme2), GTK_STYLE_PROVIDER_PRIORITY_USER);
-	gtk_style_context_reset_widgets (screen);
-
-	/* Re-sampling must see the new theme, not the palette colors the
-	 * app applied for the old theme. */
-	theme_get_xtext_colors_for_widget (label, palette, G_N_ELEMENTS (palette));
-
-	g_assert_true (stub_last_gtk_map_valid);
-	g_assert_true (gdk_rgba_parse (&expected, "#00ff00"));
-	g_assert_true (rgba_close (&stub_last_gtk_map.text_foreground, &expected));
-	g_assert_true (gdk_rgba_parse (&expected, "#ff0000"));
-	g_assert_true (rgba_close (&stub_last_gtk_map.text_background, &expected));
-
-	/* Sampling must not permanently strip the palette class. */
-	g_assert_true (gtk_style_context_has_class (gtk_widget_get_style_context (label),
-		theme_css_palette_class_name ()));
-
-	gtk_style_context_remove_provider_for_screen (screen, GTK_STYLE_PROVIDER (theme2));
-	gtk_widget_destroy (window);
-	g_object_unref (theme1);
-	g_object_unref (theme2);
-}
-
 static void
 test_access_dark_light_switch_affects_token_consumers (void)
 {
@@ -452,8 +363,6 @@ main (int argc, char **argv)
 	g_test_add_func ("/theme/access/xtext_palette_forwarding", test_access_xtext_palette_forwarding);
 	g_test_add_func ("/theme/access/xtext_palette_widget_mapping_when_gtk3_active",
 			 test_access_xtext_palette_widget_mapping_when_gtk3_active);
-	g_test_add_func ("/theme/access/palette_css_does_not_feed_back_into_sampling",
-			 test_access_palette_css_does_not_feed_back_into_sampling);
 	g_test_add_func ("/theme/access/widget_style_forwarding", test_access_widget_style_forwarding);
 	g_test_add_func ("/theme/access/dark_light_switch_affects_token_consumers",
 			 test_access_dark_light_switch_affects_token_consumers);

src/fe-gtk/theme/tests/test-theme-application-input-style.c

@@ -21,7 +21,6 @@
 #include "../../fe-gtk.h"
 
 #include "../theme-application.h"
-#include "../theme-policy.h"
 #include "../../maingui.h"
 #include "../../../common/zoitechat.h"
 #include "../../../common/zoitechatc.h"
@@ -55,19 +54,6 @@ theme_css_reload_input_style (gboolean enabled, const PangoFontDescription *font
 	css_reload_calls++;
 }
 
-gboolean
-theme_policy_system_prefers_dark (void)
-{
-	return FALSE;
-}
-
-gboolean
-theme_policy_is_dark_mode_active (unsigned int mode)
-{
-	(void) mode;
-	return FALSE;
-}
-
 void
 theme_runtime_load (void)
 {
@@ -118,9 +104,7 @@ test_invalid_font_falls_back_to_sans_11 (void)
 	g_assert_cmpint (pango_font_description_get_size (style->font_desc), ==, 11 * PANGO_SCALE);
 	g_assert_cmpint (message_calls, ==, 1);
 	g_assert_cmpint (css_reload_calls, ==, 1);
-	/* Input boxes are themed per-widget; the app-global input CSS stays
+	g_assert_true (css_enabled);
-	 * off so entries render natively under the active GTK3 theme. */
-	g_assert_false (css_enabled);
 	g_assert_nonnull (css_font_desc);
 
 	if (style->font_desc)

src/fe-gtk/theme/tests/test-theme-gtk3-settings.c

@@ -24,7 +24,6 @@
 #include <glib.h>
 #include <glib/gstdio.h>
 #include <gtk/gtk.h>
-#include <gio/gio.h>
 
 #include "../theme-gtk3.h"
 #include "../../../common/gtk3-theme-service.h"
@@ -40,14 +39,11 @@ static char *xdg_data_home;
 static char *theme_parent_root;
 static char *theme_child_root;
 static char *theme_switch_root;
-static char *theme_chrome_root;
-
-static gboolean stub_system_prefers_dark;
 
 gboolean
 theme_policy_system_prefers_dark (void)
 {
-	return stub_system_prefers_dark;
+	return FALSE;
 }
 
 static void
@@ -135,8 +131,6 @@ zoitechat_gtk3_theme_find_by_id (const char *theme_id)
 		return make_theme (theme_id, theme_child_root);
 	if (g_strcmp0 (theme_id, "switch") == 0)
 		return make_theme (theme_id, theme_switch_root);
-	if (g_strcmp0 (theme_id, "chrome") == 0)
-		return make_theme (theme_id, theme_chrome_root);
 	return NULL;
 }
 
@@ -183,11 +177,6 @@ zoitechat_gtk3_theme_build_inheritance_chain (const char *theme_root)
 		g_ptr_array_add (chain, g_strdup (theme_switch_root));
 		return chain;
 	}
-	if (g_strcmp0 (theme_root, theme_chrome_root) == 0)
-	{
-		g_ptr_array_add (chain, g_strdup (theme_chrome_root));
-		return chain;
-	}
 	g_ptr_array_unref (chain);
 	return NULL;
 }
@@ -258,22 +247,6 @@ setup_themes (void)
 		"gtk-enable-animations=false\n"
 		"gtk-cursor-blink-time=444\n");
 
-	theme_chrome_root = g_build_filename (temp_root, "chrome", NULL);
-	g_assert_cmpint (g_mkdir_with_parents (theme_chrome_root, 0700), ==, 0);
-	ensure_css_dir (theme_chrome_root, "gtk-3.24");
-	/* Defines named colors but no window/menubar/menu rules of its own,
-	 * so all chrome styling must come from the backfill provider. */
-	{
-		char *css = g_build_filename (theme_chrome_root, "gtk-3.24", "gtk.css", NULL);
-		write_file (css,
-			"@define-color theme_bg_color #e8f4e8;\n"
-			"@define-color theme_fg_color #102010;\n"
-			"@define-color theme_selected_bg_color #3465a4;\n"
-			"@define-color theme_selected_fg_color #ffffff;\n"
-			"label { color: #102010; }\n");
-		g_free (css);
-	}
-
 	path = g_build_filename (theme_parent_root, "index.theme", NULL);
 	write_file (path, "[Desktop Entry]\nName=parent\n");
 	g_free (path);
@@ -293,13 +266,11 @@ teardown_themes (void)
 	g_free (theme_parent_root);
 	g_free (theme_child_root);
 	g_free (theme_switch_root);
-	g_free (theme_chrome_root);
 	g_free (xdg_data_home);
 	g_free (temp_root);
 	theme_parent_root = NULL;
 	theme_child_root = NULL;
 	theme_switch_root = NULL;
-	theme_chrome_root = NULL;
 	xdg_data_home = NULL;
 	temp_root = NULL;
 }
@@ -372,205 +343,6 @@ test_settings_restored_on_disable_and_switch (void)
 	g_assert_false (theme_gtk3_is_active ());
 }
 
-static void
-test_follow_system_variant_resolves_to_theme_variant (void)
-{
-	GError *error = NULL;
-
-	if (!gtk_available)
-	{
-		g_test_message ("GTK display not available");
-		return;
-	}
-
-	/* An explicitly selected in-app theme must not follow the OS
-	 * light/dark preference: the legacy FOLLOW_SYSTEM variant resolves
-	 * to the variant the theme itself provides, so the window chrome
-	 * (menu bar) stays on the theme even when the system prefers dark. */
-	stub_system_prefers_dark = TRUE;
-	g_assert_true (theme_gtk3_apply ("switch", THEME_GTK3_VARIANT_FOLLOW_SYSTEM, &error));
-	g_assert_no_error (error);
-	g_assert_true (theme_gtk3_is_active ());
-	g_assert_cmpint (theme_gtk3_active_variant (), ==, THEME_GTK3_VARIANT_PREFER_LIGHT);
-	g_assert_false (get_bool_setting ("gtk-application-prefer-dark-theme"));
-
-	stub_system_prefers_dark = FALSE;
-	theme_gtk3_disable ();
-}
-
-static void
-test_menu_bar_follows_active_theme_colors (void)
-{
-	GError *error = NULL;
-	GtkWidget *window;
-	GtkWidget *menubar;
-	GtkStyleContext *context;
-	GdkRGBA bg;
-	GdkRGBA expected;
-
-	if (!gtk_available)
-	{
-		g_test_message ("GTK display not available");
-		return;
-	}
-
-	/* The chrome theme defines no menubar rules; without the backfill
-	 * provider the menu bar would fall through to the base (OS) theme
-	 * and follow the OS dark/light mode instead of the selected theme. */
-	g_assert_true (theme_gtk3_apply ("chrome", THEME_GTK3_VARIANT_PREFER_LIGHT, &error));
-	g_assert_no_error (error);
-
-	window = gtk_window_new (GTK_WINDOW_TOPLEVEL);
-	menubar = gtk_menu_bar_new ();
-	gtk_container_add (GTK_CONTAINER (window), menubar);
-	gtk_widget_show_all (window);
-
-	context = gtk_widget_get_style_context (menubar);
-	G_GNUC_BEGIN_IGNORE_DEPRECATIONS
-	gtk_style_context_get_background_color (context, GTK_STATE_FLAG_NORMAL, &bg);
-	G_GNUC_END_IGNORE_DEPRECATIONS
-	g_assert_true (gdk_rgba_parse (&expected, "#e8f4e8"));
-	g_assert_cmpfloat (ABS (bg.red - expected.red), <, 0.01);
-	g_assert_cmpfloat (ABS (bg.green - expected.green), <, 0.01);
-	g_assert_cmpfloat (ABS (bg.blue - expected.blue), <, 0.01);
-
-	/* The window background must follow the theme too, or the space
-	 * around and beside the menu bar keeps the OS theme's colors. */
-	context = gtk_widget_get_style_context (window);
-	G_GNUC_BEGIN_IGNORE_DEPRECATIONS
-	gtk_style_context_get_background_color (context, GTK_STATE_FLAG_NORMAL, &bg);
-	G_GNUC_END_IGNORE_DEPRECATIONS
-	g_assert_cmpfloat (ABS (bg.red - expected.red), <, 0.01);
-	g_assert_cmpfloat (ABS (bg.green - expected.green), <, 0.01);
-	g_assert_cmpfloat (ABS (bg.blue - expected.blue), <, 0.01);
-
-	gtk_widget_destroy (window);
-	theme_gtk3_disable ();
-}
-
-static void
-test_desktop_color_sync_does_not_leak_into_chrome (void)
-{
-	GError *error = NULL;
-	GtkWidget *window;
-	GtkWidget *menubar;
-	GtkStyleContext *context;
-	GtkCssProvider *desktop_css;
-	GdkScreen *screen;
-	GdkRGBA bg;
-	GdkRGBA expected;
-	GdkRGBA injected;
-
-	if (!gtk_available)
-	{
-		g_test_message ("GTK display not available");
-		return;
-	}
-
-	/* KDE's GTK color-scheme sync (~/.config/gtk-3.0/gtk.css) redefines
-	 * the standard named colors with the OS palette at USER priority.
-	 * The switch theme defines no named colors of its own, so a naive
-	 * @theme_bg_color reference in the chrome backfill would resolve to
-	 * the injected OS dark palette; the backfill must use its neutral
-	 * variant fallback instead. */
-	screen = gdk_screen_get_default ();
-	desktop_css = gtk_css_provider_new ();
-	gtk_css_provider_load_from_data (desktop_css,
-		"@define-color theme_bg_color #1e1e1e;\n"
-		"@define-color theme_fg_color #eeeeee;\n"
-		"menubar { background-color: #1e1e1e; }\n",
-		-1, NULL);
-	gtk_style_context_add_provider_for_screen (screen,
-		GTK_STYLE_PROVIDER (desktop_css), GTK_STYLE_PROVIDER_PRIORITY_USER);
-
-	g_assert_true (theme_gtk3_apply ("switch", THEME_GTK3_VARIANT_PREFER_LIGHT, &error));
-	g_assert_no_error (error);
-
-	{
-		GtkWidget *item;
-		GdkRGBA item_bg;
-
-		window = gtk_window_new (GTK_WINDOW_TOPLEVEL);
-		menubar = gtk_menu_bar_new ();
-		item = gtk_menu_item_new_with_label ("File");
-		gtk_menu_shell_append (GTK_MENU_SHELL (menubar), item);
-		gtk_container_add (GTK_CONTAINER (window), menubar);
-		gtk_widget_show_all (window);
-
-		context = gtk_widget_get_style_context (menubar);
-		G_GNUC_BEGIN_IGNORE_DEPRECATIONS
-		gtk_style_context_get_background_color (context, GTK_STATE_FLAG_NORMAL, &bg);
-		G_GNUC_END_IGNORE_DEPRECATIONS
-		g_assert_true (gdk_rgba_parse (&expected, "#f6f5f4"));
-		g_assert_true (gdk_rgba_parse (&injected, "#1e1e1e"));
-		g_assert_cmpfloat (ABS (bg.red - injected.red), >, 0.1);
-		g_assert_cmpfloat (ABS (bg.red - expected.red), <, 0.01);
-		g_assert_cmpfloat (ABS (bg.green - expected.green), <, 0.01);
-		g_assert_cmpfloat (ABS (bg.blue - expected.blue), <, 0.01);
-
-		/* The menu bar body and the menu items must be one surface. */
-		context = gtk_widget_get_style_context (item);
-		G_GNUC_BEGIN_IGNORE_DEPRECATIONS
-		gtk_style_context_get_background_color (context, GTK_STATE_FLAG_NORMAL, &item_bg);
-		G_GNUC_END_IGNORE_DEPRECATIONS
-		g_assert_cmpfloat (ABS (item_bg.red - bg.red), <, 0.01);
-		g_assert_cmpfloat (ABS (item_bg.green - bg.green), <, 0.01);
-		g_assert_cmpfloat (ABS (item_bg.blue - bg.blue), <, 0.01);
-	}
-
-	gtk_widget_destroy (window);
-	theme_gtk3_disable ();
-	gtk_style_context_remove_provider_for_screen (screen, GTK_STYLE_PROVIDER (desktop_css));
-	g_object_unref (desktop_css);
-}
-
-static void
-test_stale_theme_symlink_is_replaced (void)
-{
-	GError *error = NULL;
-	char *themes_dir;
-	char *link_path;
-	char *target;
-	char *active_theme_name = NULL;
-
-	if (!gtk_available)
-	{
-		g_test_message ("GTK display not available");
-		return;
-	}
-	if (!has_default_seat ())
-	{
-		g_test_message ("No default seat available");
-		return;
-	}
-
-	/* A dangling symlink left behind by a removed theme must not block
-	 * pinning gtk-theme-name to the newly applied theme. */
-	themes_dir = g_build_filename (xdg_data_home, "themes", NULL);
-	g_assert_cmpint (g_mkdir_with_parents (themes_dir, 0700), ==, 0);
-	link_path = g_build_filename (themes_dir, "switch", NULL);
-	g_unlink (link_path);
-	{
-		GFile *link_file = g_file_new_for_path (link_path);
-		g_assert_true (g_file_make_symbolic_link (link_file, "/nonexistent/switch", NULL, NULL));
-		g_object_unref (link_file);
-	}
-
-	g_assert_true (theme_gtk3_apply ("switch", THEME_GTK3_VARIANT_PREFER_LIGHT, &error));
-	g_assert_no_error (error);
-
-	target = g_file_read_link (link_path, NULL);
-	g_assert_cmpstr (target, ==, theme_switch_root);
-	g_object_get (gtk_settings_get_default (), "gtk-theme-name", &active_theme_name, NULL);
-	g_assert_cmpstr (active_theme_name, ==, "switch");
-
-	g_free (active_theme_name);
-	g_free (target);
-	g_free (link_path);
-	g_free (themes_dir);
-	theme_gtk3_disable ();
-}
-
 int
 main (int argc, char **argv)
 {
@@ -582,14 +354,6 @@ main (int argc, char **argv)
 
 	g_test_add_func ("/theme/gtk3/settings_layer_precedence", test_settings_layer_precedence);
 	g_test_add_func ("/theme/gtk3/settings_restored_on_disable_and_switch", test_settings_restored_on_disable_and_switch);
-	g_test_add_func ("/theme/gtk3/follow_system_variant_resolves_to_theme_variant",
-			 test_follow_system_variant_resolves_to_theme_variant);
-	g_test_add_func ("/theme/gtk3/menu_bar_follows_active_theme_colors",
-			 test_menu_bar_follows_active_theme_colors);
-	g_test_add_func ("/theme/gtk3/desktop_color_sync_does_not_leak_into_chrome",
-			 test_desktop_color_sync_does_not_leak_into_chrome);
-	g_test_add_func ("/theme/gtk3/stale_theme_symlink_is_replaced",
-			 test_stale_theme_symlink_is_replaced);
 
 	prefs.hex_gui_gtk3_variant = THEME_GTK3_VARIANT_PREFER_LIGHT;
 

src/fe-gtk/theme/tests/test-theme-gtk3-stub.c

@@ -89,9 +89,3 @@ theme_gtk3_is_active (void)
 {
 	return FALSE;
 }
-
-ThemeGtk3Variant
-theme_gtk3_active_variant (void)
-{
-	return THEME_GTK3_VARIANT_FOLLOW_SYSTEM;
-}

src/fe-gtk/theme/tests/test-theme-manager-auto-refresh.c

@@ -22,7 +22,6 @@
 
 #include "../theme-manager.h"
 #include "../theme-gtk3.h"
-#include "../theme-policy.h"
 #include "../../../common/zoitechat.h"
 #include "../../../common/zoitechatc.h"
 
@@ -85,15 +84,6 @@ gboolean theme_policy_system_prefers_dark (void)
 	return stub_system_prefers_dark;
 }
 
-void theme_policy_init (void)
-{
-}
-
-void theme_policy_set_appearance_changed_callback (ThemePolicyAppearanceChangedFunc callback)
-{
-	(void) callback;
-}
-
 gboolean theme_application_apply_mode (unsigned int mode, gboolean *palette_changed)
 {
 	(void) mode;
@@ -213,7 +203,6 @@ reset_state (void)
 	idle_add_calls = 0;
 	next_idle_source_id = 33;
 	prefs.hex_gui_gtk3_variant = THEME_GTK3_VARIANT_FOLLOW_SYSTEM;
-	prefs.hex_gui_gtk3_theme[0] = '\0';
 	test_theme_gtk3_stub_reset ();
 }
 
@@ -246,14 +235,12 @@ test_auto_refresh_dispatches_mode_palette_and_style_reasons (void)
 }
 
 static void
-test_auto_refresh_ignores_locked_in_app_theme (void)
+test_auto_refresh_ignores_non_auto_mode (void)
 {
 	guint listener_id;
 
 	reset_state ();
 	prefs.hex_gui_dark_mode = ZOITECHAT_DARK_MODE_DARK;
-	g_strlcpy (prefs.hex_gui_gtk3_theme, "locked-theme", sizeof (prefs.hex_gui_gtk3_theme));
-	prefs.hex_gui_gtk3_variant = THEME_GTK3_VARIANT_PREFER_LIGHT;
 	stub_apply_mode_palette_changed = TRUE;
 	listener_id = theme_listener_register ("auto.nonauto", auto_listener, NULL);
 	theme_manager_set_idle_add_func (immediate_idle_add);
@@ -267,7 +254,6 @@ test_auto_refresh_ignores_locked_in_app_theme (void)
 
 	theme_manager_set_idle_add_func (NULL);
 	theme_listener_unregister (listener_id);
-	prefs.hex_gui_gtk3_theme[0] = '\0';
 }
 
 static void
@@ -277,7 +263,6 @@ test_auto_refresh_reapplies_gtk3_for_follow_system_variant (void)
 
 	reset_state ();
 	prefs.hex_gui_dark_mode = ZOITECHAT_DARK_MODE_DARK;
-	g_strlcpy (prefs.hex_gui_gtk3_theme, "system-follower", sizeof (prefs.hex_gui_gtk3_theme));
 	prefs.hex_gui_gtk3_variant = THEME_GTK3_VARIANT_FOLLOW_SYSTEM;
 	listener_id = theme_listener_register ("auto.gtk3", auto_listener, NULL);
 	theme_manager_set_idle_add_func (immediate_idle_add);
@@ -289,39 +274,6 @@ test_auto_refresh_reapplies_gtk3_for_follow_system_variant (void)
 	g_assert_cmpint (listener_calls, ==, 0);
 	g_assert_cmpint (test_theme_gtk3_stub_apply_current_calls (), ==, 1);
 
-	theme_manager_set_idle_add_func (NULL);
-	theme_listener_unregister (listener_id);
-	prefs.hex_gui_gtk3_theme[0] = '\0';
-}
-
-static void
-test_auto_refresh_follows_system_theme_without_auto_mode (void)
-{
-	guint listener_id;
-
-	reset_state ();
-	prefs.hex_gui_dark_mode = ZOITECHAT_DARK_MODE_DARK;
-	prefs.hex_gui_gtk3_variant = THEME_GTK3_VARIANT_PREFER_LIGHT;
-	listener_id = theme_listener_register ("auto.system", auto_listener, NULL);
-	theme_manager_set_idle_add_func (immediate_idle_add);
-
-	theme_manager_refresh_auto_mode ();
-
-	/* No in-app theme is selected, so a system theme change must
-	 * re-resolve the mapped palette even outside auto dark mode. */
-	g_assert_cmpint (idle_add_calls, ==, 1);
-	g_assert_cmpint (auto_state_calls, ==, 0);
-	g_assert_cmpint (listener_calls, ==, 1);
-	g_assert_true (theme_changed_event_has_reason (&last_event, THEME_CHANGED_REASON_PALETTE));
-	g_assert_true (theme_changed_event_has_reason (&last_event, THEME_CHANGED_REASON_WIDGET_STYLE));
-	g_assert_true (theme_changed_event_has_reason (&last_event, THEME_CHANGED_REASON_USERLIST));
-	g_assert_true (theme_changed_event_has_reason (&last_event, THEME_CHANGED_REASON_MODE));
-
-	/* Unchanged system state must not dispatch again. */
-	theme_manager_refresh_auto_mode ();
-	g_assert_cmpint (idle_add_calls, ==, 2);
-	g_assert_cmpint (listener_calls, ==, 1);
-
 	theme_manager_set_idle_add_func (NULL);
 	theme_listener_unregister (listener_id);
 }
@@ -332,11 +284,9 @@ main (int argc, char **argv)
 	g_test_init (&argc, &argv, NULL);
 	g_test_add_func ("/theme/manager/auto_refresh_dispatches_mode_palette_and_style_reasons",
 			 test_auto_refresh_dispatches_mode_palette_and_style_reasons);
-	g_test_add_func ("/theme/manager/auto_refresh_ignores_locked_in_app_theme",
+	g_test_add_func ("/theme/manager/auto_refresh_ignores_non_auto_mode",
-			 test_auto_refresh_ignores_locked_in_app_theme);
+			 test_auto_refresh_ignores_non_auto_mode);
 	g_test_add_func ("/theme/manager/auto_refresh_reapplies_gtk3_for_follow_system_variant",
 			 test_auto_refresh_reapplies_gtk3_for_follow_system_variant);
-	g_test_add_func ("/theme/manager/auto_refresh_follows_system_theme_without_auto_mode",
-			 test_auto_refresh_follows_system_theme_without_auto_mode);
 	return g_test_run ();
 }

src/fe-gtk/theme/tests/test-theme-manager-dispatch-routing.c

@@ -23,7 +23,6 @@
 #include <string.h>
 
 #include "../theme-manager.h"
-#include "../theme-policy.h"
 #include "../../../common/zoitechat.h"
 #include "../../../common/zoitechatc.h"
 
@@ -78,15 +77,6 @@ gboolean theme_policy_system_prefers_dark (void)
 	return FALSE;
 }
 
-void theme_policy_init (void)
-{
-}
-
-void theme_policy_set_appearance_changed_callback (ThemePolicyAppearanceChangedFunc callback)
-{
-	(void) callback;
-}
-
 gboolean theme_application_apply_mode (unsigned int mode, gboolean *palette_changed)
 {
 	(void) mode;

src/fe-gtk/theme/tests/test-theme-manager-policy.c

@@ -22,7 +22,6 @@
 
 #include "../theme-palette.h"
 #include "../theme-manager.h"
-#include "../theme-policy.h"
 #include "../../../common/zoitechat.h"
 #include "../../../common/zoitechatc.h"
 
@@ -31,7 +30,6 @@ struct session *current_tab;
 struct session *lastact_sess;
 struct zoitechatprefs prefs;
 
-static gboolean gtk_available;
 static gboolean stub_policy_dark;
 static unsigned int stub_policy_mode;
 static gboolean stub_apply_mode_result;
@@ -81,18 +79,7 @@ void zoitechat_set_theme_post_apply_callback (zoitechat_theme_post_apply_callbac
 gboolean theme_policy_is_dark_mode_active (unsigned int mode)
 {
 	stub_policy_mode = mode;
-	if (mode == ZOITECHAT_DARK_MODE_AUTO)
 	return stub_policy_dark;
-	return fe_dark_mode_is_enabled_for (mode);
-}
-
-void theme_policy_init (void)
-{
-}
-
-void theme_policy_set_appearance_changed_callback (ThemePolicyAppearanceChangedFunc callback)
-{
-	(void) callback;
 }
 
 gboolean theme_application_apply_mode (unsigned int mode, gboolean *palette_changed)
@@ -122,12 +109,11 @@ void theme_runtime_reset_mode_colors (gboolean dark_mode)
 	(void) dark_mode;
 }
 
-gboolean theme_runtime_apply_mode (unsigned int mode, gboolean *palette_changed)
+gboolean theme_runtime_apply_mode (unsigned int mode, gboolean *dark_active)
 {
 	(void) mode;
+	(void) dark_active;
 	stub_apply_mode_calls++;
-	if (palette_changed)
-		*palette_changed = TRUE;
 	return TRUE;
 }
 
@@ -263,11 +249,10 @@ test_manager_set_token_color_routes_by_mode (void)
 	GdkRGBA color = { 0.1, 0.2, 0.3, 1.0 };
 	gboolean palette_changed = FALSE;
 
-	/* All edits target the single user palette; the mode argument is
-	 * accepted for compatibility but no longer selects a dark palette. */
 	reset_manager_stubs ();
 	stub_policy_dark = FALSE;
 	theme_manager_set_token_color (ZOITECHAT_DARK_MODE_LIGHT, THEME_TOKEN_MIRC_2, &color, &palette_changed);
+	g_assert_cmpint (stub_policy_mode, ==, ZOITECHAT_DARK_MODE_LIGHT);
 	g_assert_cmpint (stub_user_set_calls, ==, 1);
 	g_assert_cmpint (stub_dark_set_calls, ==, 0);
 	g_assert_cmpint (stub_apply_mode_calls, ==, 1);
@@ -277,14 +262,16 @@ test_manager_set_token_color_routes_by_mode (void)
 	reset_manager_stubs ();
 	stub_policy_dark = TRUE;
 	theme_manager_set_token_color (ZOITECHAT_DARK_MODE_DARK, THEME_TOKEN_MIRC_2, &color, &palette_changed);
-	g_assert_cmpint (stub_user_set_calls, ==, 1);
+	g_assert_cmpint (stub_policy_mode, ==, ZOITECHAT_DARK_MODE_DARK);
-	g_assert_cmpint (stub_dark_set_calls, ==, 0);
+	g_assert_cmpint (stub_user_set_calls, ==, 0);
+	g_assert_cmpint (stub_dark_set_calls, ==, 1);
 
 	reset_manager_stubs ();
 	stub_policy_dark = TRUE;
 	theme_manager_set_token_color (ZOITECHAT_DARK_MODE_AUTO, THEME_TOKEN_MIRC_2, &color, &palette_changed);
-	g_assert_cmpint (stub_user_set_calls, ==, 1);
+	g_assert_cmpint (stub_policy_mode, ==, ZOITECHAT_DARK_MODE_AUTO);
-	g_assert_cmpint (stub_dark_set_calls, ==, 0);
+	g_assert_cmpint (stub_user_set_calls, ==, 0);
+	g_assert_cmpint (stub_dark_set_calls, ==, 1);
 }
 
 
@@ -313,9 +300,9 @@ test_manager_set_token_color_routes_setup_indexes (void)
 		stub_policy_dark = TRUE;
 		palette_changed = FALSE;
 		theme_manager_set_token_color (ZOITECHAT_DARK_MODE_DARK, def->token, &color, &palette_changed);
-		g_assert_cmpint (stub_user_set_calls, ==, 1);
+		g_assert_cmpint (stub_dark_set_calls, ==, 1);
-		g_assert_cmpint (stub_last_user_token, ==, def->token);
+		g_assert_cmpint (stub_last_dark_token, ==, def->token);
-		g_assert_cmpint (stub_dark_set_calls, ==, 0);
+		g_assert_cmpint (stub_user_set_calls, ==, 0);
 		g_assert_true (palette_changed);
 	}
 }
@@ -358,12 +345,6 @@ test_manager_window_attach_detach_idempotence (void)
 	gulong *second_handler_ptr;
 	gulong second_handler_id;
 
-	if (!gtk_available)
-	{
-		g_test_message ("GTK display not available");
-		return;
-	}
-
 	window = gtk_window_new (GTK_WINDOW_TOPLEVEL);
 	g_assert_nonnull (window);
 
@@ -414,6 +395,5 @@ main (int argc, char **argv)
 			 test_manager_listener_registration_dispatch_and_unregister);
 	g_test_add_func ("/theme/manager/window_attach_detach_idempotence",
 			 test_manager_window_attach_detach_idempotence);
-	gtk_available = gtk_init_check (&argc, &argv);
 	return g_test_run ();
 }

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

@@ -33,7 +33,6 @@ InputStyle *input_style;
 
 static gboolean gtk_available;
 static int apply_current_calls;
-static int set_token_color_calls;
 static char applied_theme_id[256];
 static ThemeGtk3Variant applied_variant;
 static gboolean removed_selected;
@@ -105,7 +104,6 @@ theme_manager_set_token_color (unsigned int dark_mode, ThemeSemanticToken token,
         (void)dark_mode;
         (void)token;
         (void)color;
-        set_token_color_calls++;
         if (changed)
                 *changed = FALSE;
 }
@@ -311,7 +309,7 @@ theme_gtk3_is_active (void)
 #include "../theme-preferences.c"
 
 static void
-test_removed_selected_theme_falls_back_to_system_default (void)
+test_removed_selected_theme_commits_fallback_and_applies (void)
 {
         GtkWidget *page;
         theme_preferences_ui *ui;
@@ -328,26 +326,23 @@ test_removed_selected_theme_falls_back_to_system_default (void)
         g_strlcpy (prefs.hex_gui_gtk3_theme, "removed-theme", sizeof (prefs.hex_gui_gtk3_theme));
         prefs.hex_gui_gtk3_variant = THEME_GTK3_VARIANT_PREFER_DARK;
         removed_selected = FALSE;
+        apply_current_calls = 0;
+        applied_theme_id[0] = '\0';
 
         page = theme_preferences_create_page (NULL, &setup_prefs, NULL);
         ui = g_object_get_data (G_OBJECT (page), "theme-preferences-ui");
         g_assert_nonnull (ui);
 
-        apply_current_calls = 0;
-        applied_theme_id[0] = '\0';
-
         g_assert_nonnull (ui->gtk3_remove);
         gtk_button_clicked (GTK_BUTTON (ui->gtk3_remove));
 
-        /* Removing the active in-app theme returns to the system GTK3
+        g_assert_cmpstr (prefs.hex_gui_gtk3_theme, ==, "fallback-theme");
-         * theme ("None") instead of forcing another theme. */
+        g_assert_cmpstr (setup_prefs.hex_gui_gtk3_theme, ==, "fallback-theme");
-        g_assert_cmpstr (prefs.hex_gui_gtk3_theme, ==, "");
+        g_assert_cmpint (prefs.hex_gui_gtk3_variant, ==, THEME_GTK3_VARIANT_PREFER_LIGHT);
-        g_assert_cmpstr (setup_prefs.hex_gui_gtk3_theme, ==, "");
+        g_assert_cmpint (setup_prefs.hex_gui_gtk3_variant, ==, THEME_GTK3_VARIANT_PREFER_LIGHT);
-        g_assert_cmpint (prefs.hex_gui_gtk3_variant, ==, THEME_GTK3_VARIANT_FOLLOW_SYSTEM);
-        g_assert_cmpint (setup_prefs.hex_gui_gtk3_variant, ==, THEME_GTK3_VARIANT_FOLLOW_SYSTEM);
         g_assert_cmpint (apply_current_calls, ==, 1);
-        g_assert_cmpstr (applied_theme_id, ==, "");
+        g_assert_cmpstr (applied_theme_id, ==, "fallback-theme");
-        g_assert_cmpint (applied_variant, ==, THEME_GTK3_VARIANT_FOLLOW_SYSTEM);
+        g_assert_cmpint (applied_variant, ==, THEME_GTK3_VARIANT_PREFER_LIGHT);
 
         gtk_widget_destroy (page);
 }
@@ -421,55 +416,16 @@ test_select_none_resets_theme_and_applies (void)
         gtk_widget_destroy (page);
 }
 
-static void
-test_select_theme_does_not_pin_palette_tokens (void)
-{
-        GtkWidget *page;
-        theme_preferences_ui *ui;
-        struct zoitechatprefs setup_prefs;
-
-        if (!gtk_available)
-        {
-                g_test_message ("GTK display not available");
-                return;
-        }
-
-        memset (&setup_prefs, 0, sizeof (setup_prefs));
-        memset (&prefs, 0, sizeof (prefs));
-        removed_selected = FALSE;
-        apply_current_calls = 0;
-        set_token_color_calls = 0;
-
-        page = theme_preferences_create_page (NULL, &setup_prefs, NULL);
-        ui = g_object_get_data (G_OBJECT (page), "theme-preferences-ui");
-        g_assert_nonnull (ui);
-
-        /* 0 = None, 1 = removed-theme, 2 = fallback-theme */
-        gtk_combo_box_set_active (GTK_COMBO_BOX (ui->gtk3_combo), 2);
-
-        g_assert_cmpstr (prefs.hex_gui_gtk3_theme, ==, "fallback-theme");
-        g_assert_cmpint (apply_current_calls, ==, 1);
-        /* Selecting a GTK3 theme must leave the mapped palette tokens
-         * unpinned so the user list, channel tree and input box keep
-         * resolving their colors from the active theme instead of being
-         * frozen until the next restart. */
-        g_assert_cmpint (set_token_color_calls, ==, 0);
-
-        gtk_widget_destroy (page);
-}
-
 int
 main (int argc, char **argv)
 {
         g_test_init (&argc, &argv, NULL);
         gtk_available = gtk_init_check (&argc, &argv);
-        g_test_add_func ("/theme/preferences/gtk3_removed_selection_falls_back_to_system",
+        g_test_add_func ("/theme/preferences/gtk3_removed_selection_applies_fallback",
-                         test_removed_selected_theme_falls_back_to_system_default);
+                         test_removed_selected_theme_commits_fallback_and_applies);
         g_test_add_func ("/theme/preferences/gtk3_unset_keeps_system_default",
                          test_unset_theme_keeps_system_default_without_apply);
         g_test_add_func ("/theme/preferences/gtk3_select_none_resets_theme",
                          test_select_none_resets_theme_and_applies);
-        g_test_add_func ("/theme/preferences/gtk3_select_theme_does_not_pin_palette_tokens",
-                         test_select_theme_does_not_pin_palette_tokens);
         return g_test_run ();
 }

src/fe-gtk/theme/tests/test-theme-runtime-persistence.c

@@ -290,12 +290,12 @@ test_ui_edits_persist_without_legacy_array_mutation (void)
 	theme_runtime_save ();
 
 	theme_runtime_load ();
-	theme_runtime_apply_dark_mode (FALSE);
+	theme_runtime_apply_mode (ZOITECHAT_DARK_MODE_LIGHT, NULL);
 	g_assert_true (theme_runtime_get_color (THEME_TOKEN_SELECTION_FOREGROUND, &light_loaded));
 	g_assert_true (gdk_rgba_parse (&light_expected, "#224466"));
 	g_assert_true (colors_equal (&light_loaded, &light_expected));
 
-	theme_runtime_apply_dark_mode (TRUE);
+	theme_runtime_apply_mode (ZOITECHAT_DARK_MODE_DARK, NULL);
 	g_assert_true (theme_runtime_get_color (THEME_TOKEN_SELECTION_FOREGROUND, &dark_loaded));
 	g_assert_true (gdk_rgba_parse (&dark_expected, "#88aacc"));
 	g_assert_true (colors_equal (&dark_loaded, &dark_expected));

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

@@ -20,9 +20,8 @@
 
 #include "theme-access.h"
 
-#include "theme-css.h"
-#include "theme-gtk3.h"
 #include "theme-runtime.h"
+#include "theme-gtk3.h"
 
 
 
@@ -33,8 +32,7 @@ enum
 	THEME_XTEXT_MARK_BG_INDEX = 100,
 	THEME_XTEXT_FG_INDEX = 101,
 	THEME_XTEXT_BG_INDEX = 102,
-	THEME_XTEXT_MARKER_INDEX = 103,
+	THEME_XTEXT_MARKER_INDEX = 103
-	THEME_XTEXT_MARKER_LEGACY_INDEX = 36
 };
 
 static const guint8 theme_default_99_mirc_colors[THEME_XTEXT_MIRC_COLS][3] = {
@@ -113,95 +111,28 @@ theme_access_context_get_background_color (GtkStyleContext *context, GtkStateFla
 	gtk_style_context_restore (context);
 }
 
-static gboolean
-theme_access_lookup_named_color (GtkStyleContext *context, const char *const *names, GdkRGBA *out_color)
-{
-	size_t i;
-
-	/* With an in-app GTK3 theme selected, prefer the colors resolved from
-	 * the theme itself: desktop integration CSS (e.g. KDE's color-scheme
-	 * sync) redefines the same named colors with the OS palette in the
-	 * widget cascade, which would drag OS dark/light colors into the
-	 * mapped palette. */
-	for (i = 0; names[i] != NULL; i++)
-	{
-		if (theme_gtk3_lookup_theme_color (names[i], out_color))
-			return TRUE;
-	}
-
-	for (i = 0; names[i] != NULL; i++)
-	{
-		if (gtk_style_context_lookup_color (context, names[i], out_color))
-			return TRUE;
-	}
-
-	return FALSE;
-}
-
-static void
-theme_access_resolve_map_background (GtkStyleContext *context, GtkStateFlags state,
-				     const char *const *named_fallbacks, GdkRGBA *out_color)
-{
-	GdkRGBA named;
-
-	theme_access_context_get_background_color (context, state, out_color);
-	if (out_color->alpha > 0.0)
-		return;
-
-	/* Unstyled CSS nodes (e.g. the custom chat text widget) report a fully
-	 * transparent background; fall back to the theme's named colors so the
-	 * mapped palette still follows the active GTK3 theme. */
-	if (theme_access_lookup_named_color (context, named_fallbacks, &named))
-		*out_color = named;
-}
-
 static gboolean
 theme_access_get_gtk_palette_map (GtkWidget *widget, ThemeGtkPaletteMap *out_map)
 {
-	static const char *const base_fallbacks[] = { "theme_base_color", "theme_bg_color", NULL };
-	static const char *const selected_bg_fallbacks[] = { "theme_selected_bg_color", NULL };
-	static const char *const selected_fg_fallbacks[] = { "theme_selected_fg_color", NULL };
 	GtkStyleContext *context;
-	const char *palette_class;
-	gboolean had_palette_class;
-	GdkRGBA named;
 	GdkRGBA accent;
 
 	g_return_val_if_fail (out_map != NULL, FALSE);
-	if (widget == NULL || !GTK_IS_WIDGET (widget))
+	if (!theme_gtk3_is_active () || widget == NULL || !GTK_IS_WIDGET (widget))
 		return FALSE;
 
 	context = gtk_widget_get_style_context (widget);
 	if (context == NULL)
 		return FALSE;
 
-	/* The application paints these widgets with fixed colors through the
-	 * zoitechat-palette provider.  Sampling must see the underlying GTK3
-	 * theme, not the colors we applied earlier: otherwise every theme
-	 * change reads back the previous palette and the user list, channel
-	 * tree and input box keep their stale colors until a restart. */
-	palette_class = theme_css_palette_class_name ();
-	had_palette_class = gtk_style_context_has_class (context, palette_class);
-	if (had_palette_class)
-		gtk_style_context_remove_class (context, palette_class);
-
 	theme_access_context_get_color (context, GTK_STATE_FLAG_NORMAL, &out_map->text_foreground);
-	theme_access_resolve_map_background (context, GTK_STATE_FLAG_NORMAL, base_fallbacks,
+	theme_access_context_get_background_color (context, GTK_STATE_FLAG_NORMAL, &out_map->text_background);
-					     &out_map->text_background);
 	theme_access_context_get_color (context, GTK_STATE_FLAG_SELECTED, &out_map->selection_foreground);
-	theme_access_resolve_map_background (context, GTK_STATE_FLAG_SELECTED, selected_bg_fallbacks,
+	theme_access_context_get_background_color (context, GTK_STATE_FLAG_SELECTED, &out_map->selection_background);
-					     &out_map->selection_background);
-	if (gdk_rgba_equal (&out_map->selection_foreground, &out_map->text_foreground) &&
-	    theme_access_lookup_named_color (context, selected_fg_fallbacks, &named))
-		out_map->selection_foreground = named;
 	theme_access_context_get_color (context, GTK_STATE_FLAG_LINK, &accent);
 	if (accent.alpha <= 0.0)
 		accent = out_map->selection_background;
 	out_map->accent = accent;
-
-	if (had_palette_class)
-		gtk_style_context_add_class (context, palette_class);
-
 	out_map->enabled = TRUE;
 	return TRUE;
 }
@@ -292,35 +223,14 @@ theme_get_xtext_colors (XTextColor *palette, size_t palette_len)
 void
 theme_get_xtext_colors_for_widget (GtkWidget *widget, XTextColor *palette, size_t palette_len)
 {
-	ThemeGtkPaletteMap gtk_map = { 0 };
 	ThemeWidgetStyleValues style_values;
 	GdkRGBA marker_color;
-	gboolean have_marker = FALSE;
 
 	if (!palette)
 		return;
 
-	if (theme_access_get_gtk_palette_map (widget, &gtk_map))
+	theme_get_widget_style_values_for_widget (widget, &style_values);
-	{
-		theme_runtime_get_widget_style_values_mapped (&gtk_map, &style_values);
-		theme_runtime_get_xtext_colors_mapped (&gtk_map, palette, palette_len);
-	}
-	else
-	{
-		theme_runtime_get_widget_style_values (&style_values);
 	theme_runtime_get_xtext_colors (palette, palette_len);
-	}
-	if (palette_len > THEME_XTEXT_MARKER_LEGACY_INDEX)
-	{
-		/* The marker token lives at its legacy slot before the extended
-		 * mIRC palette overwrites it; keep the (possibly theme-mapped)
-		 * value for the dedicated marker index below. */
-		marker_color.red = palette[THEME_XTEXT_MARKER_LEGACY_INDEX].red;
-		marker_color.green = palette[THEME_XTEXT_MARKER_LEGACY_INDEX].green;
-		marker_color.blue = palette[THEME_XTEXT_MARKER_LEGACY_INDEX].blue;
-		marker_color.alpha = palette[THEME_XTEXT_MARKER_LEGACY_INDEX].alpha;
-		have_marker = TRUE;
-	}
 	if (palette_len >= THEME_XTEXT_MIRC_COLS)
 		theme_access_apply_default_99_palette (palette, palette_len);
 	if (palette_len > THEME_XTEXT_MARK_FG_INDEX)
@@ -339,7 +249,7 @@ theme_get_xtext_colors_for_widget (GtkWidget *widget, XTextColor *palette, size_
 	}
 	if (palette_len > THEME_XTEXT_MARKER_INDEX)
 	{
-		if (!have_marker && !theme_runtime_get_color (THEME_TOKEN_MARKER, &marker_color))
+		if (!theme_runtime_get_color (THEME_TOKEN_MARKER, &marker_color))
 			marker_color = style_values.selection_background;
 		palette[THEME_XTEXT_MARKER_INDEX].red = marker_color.red;
 		palette[THEME_XTEXT_MARKER_INDEX].green = marker_color.green;

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

@@ -25,7 +25,6 @@
 #include "theme-css.h"
 #include "theme-runtime.h"
 #include "theme-gtk3.h"
-#include "theme-policy.h"
 #include "../maingui.h"
 #include <gtk/gtk.h>
 
@@ -56,19 +55,10 @@ theme_application_apply_toplevel_theme (gboolean dark)
 
 	if (theme_gtk3_is_active ())
 	{
-		/* An explicitly selected in-app theme controls its own
+		if (prefs.hex_gui_gtk3_variant == THEME_GTK3_VARIANT_PREFER_DARK)
-		 * variant; the OS light/dark preference must not leak into
+			prefer_dark = TRUE;
-		 * the window chrome (menu bar, dialogs).  The active variant
+		else if (prefs.hex_gui_gtk3_variant == THEME_GTK3_VARIANT_PREFER_LIGHT)
-		 * is always resolved to light or dark. */
+			prefer_dark = FALSE;
-		prefer_dark = theme_gtk3_active_variant () == THEME_GTK3_VARIANT_PREFER_DARK;
-	}
-	else
-	{
-		/* Following the system GTK3 theme: only request the dark
-		 * variant when the dark-mode preference (or the system, in
-		 * auto mode) asks for it, instead of clobbering the value
-		 * that came from the user's GTK configuration. */
-		prefer_dark = theme_policy_is_dark_mode_active (prefs.hex_gui_dark_mode);
 	}
 
 	if (settings && g_object_class_find_property (G_OBJECT_GET_CLASS (settings),

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

@@ -29,6 +29,8 @@
 static const char *theme_css_selector_input = "#zoitechat-inputbox";
 static const char *theme_css_selector_input_text = "#zoitechat-inputbox text";
 static const char *theme_css_selector_palette_class = "zoitechat-palette";
+static const char *theme_css_selector_dark_class = "zoitechat-dark";
+static const char *theme_css_selector_light_class = "zoitechat-light";
 static const char *theme_css_palette_provider_key = "zoitechat-palette-provider";
 static const guint theme_css_provider_priority = GTK_STYLE_PROVIDER_PRIORITY_USER;
 static const char *theme_css_palette_scope_selectors =
@@ -60,12 +62,6 @@ typedef struct
 static GtkCssProvider *theme_css_input_provider;
 static ThemeCssInputFingerprint theme_css_input_fp;
 
-const char *
-theme_css_palette_class_name (void)
-{
-	return theme_css_selector_palette_class;
-}
-
 void
 theme_css_apply_app_provider (GtkStyleProvider *provider)
 {
@@ -366,9 +362,61 @@ theme_css_apply_palette_widget (GtkWidget *widget, const GdkRGBA *bg, const GdkR
 char *
 theme_css_build_toplevel_classes (void)
 {
-	/* Windows, dialogs and menus must render with the colors of the active
+	return g_strdup_printf (
-	 * GTK3 theme (system or in-app selected).  zoitechat-dark and
+		"window.%s, window.%s:backdrop, .%s {"
-	 * zoitechat-light are marker classes only; painting fixed colors here
+		"background-color: #202020;"
-	 * overrides every theme (issue #334). */
+		"color: #f0f0f0;"
-	return g_strdup ("");
+		"border-color: #202020;"
+		"}"
+		"window.%s menubar, window.%s menubar:backdrop, window.%s menubar box, window.%s menubar box:backdrop, window.%s menuitem, window.%s menuitem:backdrop {"
+		"background-color: @theme_bg_color;"
+		"background-image: none;"
+		"color: @theme_fg_color;"
+		"border-color: @theme_bg_color;"
+		"}"
+		"window.%s menuitem:hover, window.%s menuitem:selected {"
+		"background-color: @theme_selected_bg_color;"
+		"background-image: none;"
+		"color: @theme_selected_fg_color;"
+		"border-color: @theme_selected_bg_color;"
+		"}"
+		"window.%s, window.%s:backdrop, .%s {"
+		"background-color: #f6f6f6;"
+		"color: #101010;"
+		"border-color: #f6f6f6;"
+		"}"
+		"window.%s menubar, window.%s menubar:backdrop, window.%s menubar box, window.%s menubar box:backdrop, window.%s menuitem, window.%s menuitem:backdrop {"
+		"background-color: @theme_bg_color;"
+		"background-image: none;"
+		"color: @theme_fg_color;"
+		"border-color: @theme_bg_color;"
+		"}"
+		"window.%s menuitem:hover, window.%s menuitem:selected {"
+		"background-color: @theme_selected_bg_color;"
+		"background-image: none;"
+		"color: @theme_selected_fg_color;"
+		"border-color: @theme_selected_bg_color;"
+		"}",
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_dark_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class,
+		theme_css_selector_light_class);
 }

src/fe-gtk/theme/theme-css.h

@@ -30,7 +30,6 @@
  * theme_css_apply_widget_provider:
  * Use for widget-local CSS providers attached to a specific widget context.
  */
-const char *theme_css_palette_class_name (void);
 void theme_css_apply_app_provider (GtkStyleProvider *provider);
 void theme_css_remove_app_provider (GtkStyleProvider *provider);
 void theme_css_apply_widget_provider (GtkWidget *widget, GtkStyleProvider *provider);

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

@@ -33,60 +33,11 @@
 
 static GPtrArray *theme_gtk3_providers_base;
 static GPtrArray *theme_gtk3_providers_variant;
-static GtkCssProvider *theme_gtk3_chrome_provider;
-static GHashTable *theme_gtk3_theme_colors;
 static GHashTable *theme_gtk3_provider_cache;
 static gboolean theme_gtk3_active;
 static char *theme_gtk3_current_id;
 static ThemeGtk3Variant theme_gtk3_current_variant;
 
-/* Menu bar and menus fall back to the base gtk-theme-name theme (the OS
- * theme) for any property the selected in-app theme's CSS does not define,
- * which mixes OS dark/light colors into the chrome (issue: dark menu bar on
- * a light in-app theme when the OS is in dark mode).  While an in-app theme
- * is active, backfill those widgets with the theme's own named colors.  The
- * provider sits just below the theme providers, so a theme that styles its
- * menus keeps full control. */
-#define THEME_GTK3_CHROME_PRIORITY (GTK_STYLE_PROVIDER_PRIORITY_USER - 1)
-/* Concrete color values are formatted in: desktop integration CSS (e.g.
- * KDE's color-scheme sync in ~/.config/gtk-3.0/gtk.css) redefines the
- * standard named colors with the OS palette at USER priority, so a
- * "@theme_bg_color" reference here would resolve to the OS dark/light
- * colors instead of the selected theme's. */
-static const char theme_gtk3_chrome_css_template[] =
-	".background, .background:backdrop {"
-	" background-color: %s;"
-	" color: %s;"
-	"}"
-	"headerbar, headerbar:backdrop, .titlebar, .titlebar:backdrop {"
-	" background-color: %s;"
-	" background-image: none;"
-	" color: %s;"
-	"}";
-
-/* The menu bar body and the menu items are separate CSS surfaces; desktop
- * integration CSS or a theme can style one but not the other, splitting
- * the menu bar row into two colors.  These rules sit above the theme and
- * the desktop CSS so body and items always render as one surface. */
-#define THEME_GTK3_MENU_UNIFORM_PRIORITY (GTK_STYLE_PROVIDER_PRIORITY_USER + 40)
-static const char theme_gtk3_menu_uniform_template[] =
-	"menubar, menubar:backdrop, menubar > menuitem, menubar > menuitem:backdrop,"
-	"menu, menu:backdrop, .menu, .menu:backdrop, menu > menuitem, menu > menuitem:backdrop {"
-	" background-color: %s;"
-	" background-image: none;"
-	" color: %s;"
-	"}"
-	"menubar > menuitem:hover, menubar > menuitem:selected,"
-	"menu > menuitem:hover, menu > menuitem:selected,"
-	".menu menuitem:hover, .menu menuitem:selected {"
-	" background-color: %s;"
-	" background-image: none;"
-	" color: %s;"
-	"}"
-	"menubar > menuitem:disabled, menu > menuitem:disabled {"
-	" color: alpha(%s, 0.5);"
-	"}";
-
 typedef struct
 {
 	GHashTable *defaults;
@@ -98,7 +49,6 @@ typedef struct
 static ThemeGtk3SettingsState theme_gtk3_settings_state;
 
 static gboolean settings_apply_property (GtkSettings *settings, const char *property_name, const char *raw_value);
-static void theme_gtk3_menu_uniform_apply (const char *bg, const char *fg, const char *sel_bg, const char *sel_fg);
 
 static gboolean
 theme_gtk3_theme_name_is_dark (const char *name)
@@ -449,21 +399,6 @@ settings_theme_link_search_path (const char *theme_root, const char *theme_name)
 	}
 
 	link_path = g_build_filename (themes_root, theme_name, NULL);
-
-	/* A leftover symlink from a removed or relocated theme would satisfy
-	 * the EXISTS check while pointing at the wrong (or no) theme, leaving
-	 * the base gtk-theme-name on the OS theme.  Replace it. */
-	{
-		char *link_target = g_file_read_link (link_path, NULL);
-		if (link_target)
-		{
-			if (g_strcmp0 (link_target, theme_root) != 0 ||
-			    !g_file_test (link_path, G_FILE_TEST_IS_DIR))
-				g_unlink (link_path);
-			g_free (link_target);
-		}
-	}
-
 	if (!g_file_test (link_path, G_FILE_TEST_EXISTS))
 	{
 		GFile *link_file = g_file_new_for_path (link_path);
@@ -822,176 +757,12 @@ settings_apply_from_file (const char *theme_root, const char *css_dir)
 	g_ptr_array_unref (settings_paths);
 }
 
-static gboolean
-theme_gtk3_providers_define_color (const char *name)
-{
-	char *needle = g_strdup_printf ("@define-color %s ", name);
-	gboolean found = FALSE;
-	guint i;
-
-	for (i = 0; !found && theme_gtk3_providers_base && i < theme_gtk3_providers_base->len; i++)
-	{
-		char *dump = gtk_css_provider_to_string (g_ptr_array_index (theme_gtk3_providers_base, i));
-		found = dump && strstr (dump, needle) != NULL;
-		g_free (dump);
-	}
-	for (i = 0; !found && theme_gtk3_providers_variant && i < theme_gtk3_providers_variant->len; i++)
-	{
-		char *dump = gtk_css_provider_to_string (g_ptr_array_index (theme_gtk3_providers_variant, i));
-		found = dump && strstr (dump, needle) != NULL;
-		g_free (dump);
-	}
-
-	g_free (needle);
-	return found;
-}
-
-/* Resolve a named color from the selected theme itself.  The lookup
- * context layers the theme's providers over the screen cascade, so the
- * theme's own @define-color wins; when the theme does not define the
- * name at all, fall back to a neutral palette matching the theme's
- * variant instead of whatever the desktop injected into the cascade. */
-static char *
-theme_gtk3_chrome_resolve_color (GtkStyleContext *lookup, const char *name, const char *fallback)
-{
-	GdkRGBA color;
-
-	if (!theme_gtk3_providers_define_color (name) ||
-	    !gtk_style_context_lookup_color (lookup, name, &color))
-		g_assert (gdk_rgba_parse (&color, fallback));
-
-	if (!theme_gtk3_theme_colors)
-		theme_gtk3_theme_colors = g_hash_table_new_full (g_str_hash, g_str_equal, g_free, g_free);
-	{
-		GdkRGBA *stored = g_new (GdkRGBA, 1);
-		*stored = color;
-		g_hash_table_insert (theme_gtk3_theme_colors, g_strdup (name), stored);
-	}
-
-	return gdk_rgba_to_string (&color);
-}
-
-gboolean
-theme_gtk3_lookup_theme_color (const char *name, GdkRGBA *out_color)
-{
-	GdkRGBA *stored;
-
-	if (!theme_gtk3_active || !theme_gtk3_theme_colors || !name || !out_color)
-		return FALSE;
-
-	stored = g_hash_table_lookup (theme_gtk3_theme_colors, name);
-	if (!stored)
-		return FALSE;
-
-	*out_color = *stored;
-	return TRUE;
-}
-
-static void
-theme_gtk3_chrome_provider_apply (gboolean prefer_dark)
-{
-	GdkScreen *screen = gdk_screen_get_default ();
-	GtkStyleContext *lookup;
-	char *bg;
-	char *fg;
-	char *sel_bg;
-	char *sel_fg;
-	char *css;
-	guint i;
-
-	if (!screen)
-		return;
-
-	lookup = gtk_style_context_new ();
-	for (i = 0; theme_gtk3_providers_base && i < theme_gtk3_providers_base->len; i++)
-		gtk_style_context_add_provider (lookup,
-			GTK_STYLE_PROVIDER (g_ptr_array_index (theme_gtk3_providers_base, i)),
-			GTK_STYLE_PROVIDER_PRIORITY_USER);
-	for (i = 0; theme_gtk3_providers_variant && i < theme_gtk3_providers_variant->len; i++)
-		gtk_style_context_add_provider (lookup,
-			GTK_STYLE_PROVIDER (g_ptr_array_index (theme_gtk3_providers_variant, i)),
-			GTK_STYLE_PROVIDER_PRIORITY_USER);
-
-	if (theme_gtk3_theme_colors)
-		g_hash_table_remove_all (theme_gtk3_theme_colors);
-
-	bg = theme_gtk3_chrome_resolve_color (lookup, "theme_bg_color",
-					      prefer_dark ? "#242424" : "#f6f5f4");
-	fg = theme_gtk3_chrome_resolve_color (lookup, "theme_fg_color",
-					      prefer_dark ? "#eeeeee" : "#2e3436");
-	sel_bg = theme_gtk3_chrome_resolve_color (lookup, "theme_selected_bg_color", "#3584e4");
-	sel_fg = theme_gtk3_chrome_resolve_color (lookup, "theme_selected_fg_color", "#ffffff");
-	/* Not used by the chrome CSS itself, but resolved and stored for the
-	 * mapped-palette sampling fallback (chat area and friends). */
-	g_free (theme_gtk3_chrome_resolve_color (lookup, "theme_base_color",
-						 prefer_dark ? "#1e1e1e" : "#ffffff"));
-	g_object_unref (lookup);
-
-	css = g_strdup_printf (theme_gtk3_chrome_css_template,
-			       bg, fg,
-			       bg, fg);
-
-	if (!theme_gtk3_chrome_provider)
-		theme_gtk3_chrome_provider = gtk_css_provider_new ();
-	gtk_css_provider_load_from_data (theme_gtk3_chrome_provider, css, -1, NULL);
-
-	gtk_style_context_add_provider_for_screen (screen,
-		GTK_STYLE_PROVIDER (theme_gtk3_chrome_provider),
-		THEME_GTK3_CHROME_PRIORITY);
-
-	theme_gtk3_menu_uniform_apply (bg, fg, sel_bg, sel_fg);
-
-	g_free (css);
-	g_free (sel_fg);
-	g_free (sel_bg);
-	g_free (fg);
-	g_free (bg);
-}
-
-static GtkCssProvider *theme_gtk3_menu_uniform_provider;
-
-static void
-theme_gtk3_menu_uniform_apply (const char *bg, const char *fg, const char *sel_bg, const char *sel_fg)
-{
-	GdkScreen *screen = gdk_screen_get_default ();
-	char *css;
-
-	if (!screen)
-		return;
-
-	css = g_strdup_printf (theme_gtk3_menu_uniform_template, bg, fg, sel_bg, sel_fg, fg);
-	if (!theme_gtk3_menu_uniform_provider)
-		theme_gtk3_menu_uniform_provider = gtk_css_provider_new ();
-	gtk_css_provider_load_from_data (theme_gtk3_menu_uniform_provider, css, -1, NULL);
-	gtk_style_context_add_provider_for_screen (screen,
-		GTK_STYLE_PROVIDER (theme_gtk3_menu_uniform_provider),
-		THEME_GTK3_MENU_UNIFORM_PRIORITY);
-	g_free (css);
-}
-
-static void
-theme_gtk3_chrome_provider_remove (void)
-{
-	GdkScreen *screen = gdk_screen_get_default ();
-
-	if (!screen)
-		return;
-
-	if (theme_gtk3_chrome_provider)
-		gtk_style_context_remove_provider_for_screen (screen,
-			GTK_STYLE_PROVIDER (theme_gtk3_chrome_provider));
-	if (theme_gtk3_menu_uniform_provider)
-		gtk_style_context_remove_provider_for_screen (screen,
-			GTK_STYLE_PROVIDER (theme_gtk3_menu_uniform_provider));
-}
-
 static void
 theme_gtk3_remove_provider (void)
 {
 	GdkScreen *screen = gdk_screen_get_default ();
 	guint i;
 
-	theme_gtk3_chrome_provider_remove ();
 	if (screen && theme_gtk3_providers_variant)
 	{
 		for (i = 0; i < theme_gtk3_providers_variant->len; i++)
@@ -1095,7 +866,6 @@ load_css_with_variant (ZoitechatGtk3Theme *theme, ThemeGtk3Variant variant, GErr
 	}
 
 	g_ptr_array_unref (chain);
-	theme_gtk3_chrome_provider_apply (prefer_dark);
 	settings_apply_for_variant (variant);
 	theme_gtk3_reset_widgets ();
 	theme_gtk3_active = TRUE;
@@ -1106,34 +876,23 @@ static gboolean
 theme_gtk3_apply_internal (const char *theme_id, ThemeGtk3Variant variant, gboolean force_reload, GError **error)
 {
 	ZoitechatGtk3Theme *theme;
-	char *previous_id;
+	char *previous_id = g_strdup (theme_gtk3_current_id);
-	ThemeGtk3Variant previous_variant;
+	ThemeGtk3Variant previous_variant = theme_gtk3_current_variant;
-	gboolean had_previous;
+	gboolean had_previous = theme_gtk3_active && previous_id && previous_id[0];
 	gboolean ok;
 
-	theme = zoitechat_gtk3_theme_find_by_id (theme_id);
-	if (!theme)
-		return g_set_error_literal (error, G_FILE_ERROR, G_FILE_ERROR_NOENT, "GTK3 theme not found."), FALSE;
-
-	/* An explicitly selected in-app theme pins the rendering to that
-	 * theme: it must not flip with the OS light/dark preference.  Legacy
-	 * configs still carry FOLLOW_SYSTEM (the pre-inference default), so
-	 * resolve it to the variant the theme itself provides. */
-	if (variant == THEME_GTK3_VARIANT_FOLLOW_SYSTEM)
-		variant = theme_gtk3_infer_variant (theme);
-
 	if (!force_reload &&
 		theme_gtk3_active &&
 		g_strcmp0 (theme_gtk3_current_id, theme_id) == 0 &&
 		theme_gtk3_current_variant == variant)
-	{
-		zoitechat_gtk3_theme_free (theme);
 		return TRUE;
-	}
 
-	previous_id = g_strdup (theme_gtk3_current_id);
+	theme = zoitechat_gtk3_theme_find_by_id (theme_id);
-	previous_variant = theme_gtk3_current_variant;
+	if (!theme)
-	had_previous = theme_gtk3_active && previous_id && previous_id[0];
+	{
+		g_free (previous_id);
+		return g_set_error_literal (error, G_FILE_ERROR, G_FILE_ERROR_NOENT, "GTK3 theme not found."), FALSE;
+	}
 
 	theme_gtk3_remove_provider ();
 	if (force_reload)
@@ -1202,12 +961,6 @@ void
 theme_gtk3_disable (void)
 {
 	theme_gtk3_remove_provider ();
-	g_clear_object (&theme_gtk3_chrome_provider);
-	g_clear_pointer (&theme_gtk3_theme_colors, g_hash_table_destroy);
-	/* Following the system theme: keep the menu bar body and the menu
-	 * items on one surface too, using the system's own colors. */
-	theme_gtk3_menu_uniform_apply ("@theme_bg_color", "@theme_fg_color",
-				       "@theme_selected_bg_color", "@theme_selected_fg_color");
 	g_clear_pointer (&theme_gtk3_current_id, g_free);
 	theme_gtk3_invalidate_provider_cache ();
 	g_clear_pointer (&theme_gtk3_provider_cache, g_hash_table_destroy);
@@ -1237,13 +990,3 @@ theme_gtk3_is_active (void)
 {
 	return theme_gtk3_active;
 }
-
-ThemeGtk3Variant
-theme_gtk3_active_variant (void)
-{
-	/* Always resolved to PREFER_LIGHT or PREFER_DARK while a theme is
-	 * active; FOLLOW_SYSTEM only when no in-app theme is applied. */
-	if (!theme_gtk3_active)
-		return THEME_GTK3_VARIANT_FOLLOW_SYSTEM;
-	return theme_gtk3_current_variant;
-}

src/fe-gtk/theme/theme-gtk3.h

@@ -22,7 +22,6 @@
 #define ZOITECHAT_THEME_GTK3_H
 
 #include <glib.h>
-#include <gdk/gdk.h>
 
 typedef enum
 {
@@ -39,7 +38,5 @@ ThemeGtk3Variant theme_gtk3_variant_for_theme (const char *theme_id);
 void theme_gtk3_invalidate_provider_cache (void);
 void theme_gtk3_disable (void);
 gboolean theme_gtk3_is_active (void);
-ThemeGtk3Variant theme_gtk3_active_variant (void);
-gboolean theme_gtk3_lookup_theme_color (const char *name, GdkRGBA *out_color);
 
 #endif

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

@@ -58,52 +58,10 @@ typedef struct
 	gboolean resolved_dark_preference;
 	char gtk3_theme_id[sizeof prefs.hex_gui_gtk3_theme];
 	int gtk3_variant;
-	char *gtk_theme_name;
 } ThemeManagerAutoRefreshCache;
 
 static ThemeManagerAutoRefreshCache theme_manager_auto_refresh_cache;
 
-static void
-theme_manager_auto_refresh_cache_reset (void)
-{
-	g_free (theme_manager_auto_refresh_cache.gtk_theme_name);
-	memset (&theme_manager_auto_refresh_cache, 0, sizeof (theme_manager_auto_refresh_cache));
-}
-
-static void
-theme_manager_auto_refresh_cache_store (gboolean resolved_dark_preference, char *gtk_theme_name)
-{
-	theme_manager_auto_refresh_cache.initialized = TRUE;
-	theme_manager_auto_refresh_cache.resolved_dark_preference = resolved_dark_preference;
-	theme_manager_auto_refresh_cache.gtk3_variant = prefs.hex_gui_gtk3_variant;
-	g_strlcpy (theme_manager_auto_refresh_cache.gtk3_theme_id,
-		   prefs.hex_gui_gtk3_theme,
-		   sizeof (theme_manager_auto_refresh_cache.gtk3_theme_id));
-	g_free (theme_manager_auto_refresh_cache.gtk_theme_name);
-	theme_manager_auto_refresh_cache.gtk_theme_name = gtk_theme_name;
-}
-
-static char *
-theme_manager_dup_gtk_theme_name (void)
-{
-	GtkSettings *settings = gtk_settings_get_default ();
-	char *gtk_theme_name = NULL;
-
-	if (settings)
-		g_object_get (settings, "gtk-theme-name", &gtk_theme_name, NULL);
-	return gtk_theme_name;
-}
-
-/* Record the state the application started up with so that the first queued
- * auto refresh only reacts to real changes; startup already applied the
- * current theme, and consumers such as the tray are not ready yet. */
-static void
-theme_manager_auto_refresh_cache_prime (void)
-{
-	theme_manager_auto_refresh_cache_store (theme_policy_system_prefers_dark (),
-						theme_manager_dup_gtk_theme_name ());
-}
-
 static void theme_manager_apply_platform_window_theme (GtkWidget *window);
 
 static void
@@ -187,9 +145,7 @@ theme_manager_auto_dark_mode_changed (GtkSettings *settings, GParamSpec *pspec,
 	gboolean color_change = FALSE;
 	gboolean should_refresh_gtk3;
 	gboolean gtk3_refresh;
-	gboolean follows_system_theme;
 	gboolean resolved_dark_preference;
-	char *gtk_theme_name = NULL;
 	static gboolean in_handler = FALSE;
 
 	(void) settings;
@@ -198,47 +154,39 @@ theme_manager_auto_dark_mode_changed (GtkSettings *settings, GParamSpec *pspec,
 
 	resolved_dark_preference = theme_policy_system_prefers_dark ();
 	gtk3_refresh = theme_manager_should_refresh_gtk3 ();
-	follows_system_theme = prefs.hex_gui_gtk3_theme[0] == '\0';
 	should_refresh_gtk3 = gtk3_refresh || prefs.hex_gui_dark_mode == ZOITECHAT_DARK_MODE_AUTO;
 
-	gtk_theme_name = theme_manager_dup_gtk_theme_name ();
-
 	if (theme_manager_auto_refresh_cache.initialized &&
 	    theme_manager_auto_refresh_cache.resolved_dark_preference == resolved_dark_preference &&
 	    theme_manager_auto_refresh_cache.gtk3_variant == prefs.hex_gui_gtk3_variant &&
-	    g_strcmp0 (theme_manager_auto_refresh_cache.gtk3_theme_id, prefs.hex_gui_gtk3_theme) == 0 &&
+	    g_strcmp0 (theme_manager_auto_refresh_cache.gtk3_theme_id, prefs.hex_gui_gtk3_theme) == 0)
-	    g_strcmp0 (theme_manager_auto_refresh_cache.gtk_theme_name, gtk_theme_name) == 0)
-	{
-		g_free (gtk_theme_name);
 		return;
-	}
 
-	theme_manager_auto_refresh_cache_store (resolved_dark_preference, gtk_theme_name);
+	theme_manager_auto_refresh_cache.initialized = TRUE;
+	theme_manager_auto_refresh_cache.resolved_dark_preference = resolved_dark_preference;
+	theme_manager_auto_refresh_cache.gtk3_variant = prefs.hex_gui_gtk3_variant;
+	g_strlcpy (theme_manager_auto_refresh_cache.gtk3_theme_id,
+		   prefs.hex_gui_gtk3_theme,
+		   sizeof (theme_manager_auto_refresh_cache.gtk3_theme_id));
 
-	if (prefs.hex_gui_dark_mode != ZOITECHAT_DARK_MODE_AUTO && !gtk3_refresh && !follows_system_theme)
+	if (prefs.hex_gui_dark_mode != ZOITECHAT_DARK_MODE_AUTO && !gtk3_refresh)
 		return;
 	if (in_handler)
 		return;
 
 	in_handler = TRUE;
 
-	if (prefs.hex_gui_dark_mode == ZOITECHAT_DARK_MODE_AUTO || follows_system_theme)
-	{
-		theme_manager_commit_preferences (prefs.hex_gui_dark_mode, &color_change);
 	if (prefs.hex_gui_dark_mode == ZOITECHAT_DARK_MODE_AUTO)
+	{
 		fe_set_auto_dark_mode_state (resolved_dark_preference);
+		theme_manager_commit_preferences (prefs.hex_gui_dark_mode, &color_change);
+		if (color_change)
+			theme_manager_dispatch_changed (THEME_CHANGED_REASON_PALETTE | THEME_CHANGED_REASON_WIDGET_STYLE | THEME_CHANGED_REASON_USERLIST | THEME_CHANGED_REASON_MODE);
 	}
 
 	if (should_refresh_gtk3)
 		theme_gtk3_apply_current (NULL);
 
-	if (prefs.hex_gui_dark_mode == ZOITECHAT_DARK_MODE_AUTO || follows_system_theme)
-	{
-		/* The system theme feeds the mapped palette (chat area, user
-		 * list, channel tree), so consumers must re-resolve colors. */
-		theme_manager_dispatch_changed (THEME_CHANGED_REASON_PALETTE | THEME_CHANGED_REASON_WIDGET_STYLE | THEME_CHANGED_REASON_USERLIST | THEME_CHANGED_REASON_MODE);
-	}
-
 	in_handler = FALSE;
 }
 
@@ -256,28 +204,18 @@ theme_manager_run_auto_refresh (gpointer data)
 static void
 theme_manager_queue_auto_refresh (GtkSettings *settings, GParamSpec *pspec, gpointer data)
 {
-	guint source_id;
-
 	(void) settings;
 	(void) pspec;
 
 	if (theme_manager_auto_refresh_source != 0)
 		return;
 
-	theme_manager_auto_refresh_source = 1;
+	theme_manager_auto_refresh_source = theme_manager_idle_add_func (theme_manager_run_auto_refresh, data);
-	source_id = theme_manager_idle_add_func (theme_manager_run_auto_refresh, data);
-	/* An injected idle-add may run the refresh synchronously and already
-	 * have cleared the pending marker; only keep the id while pending. */
-	if (theme_manager_auto_refresh_source != 0)
-		theme_manager_auto_refresh_source = source_id;
 }
 
 void
 theme_manager_init (void)
 {
-	static gboolean settings_signals_connected = FALSE;
-	GtkSettings *settings;
-
 	if (!theme_manager_listeners)
 		theme_manager_listeners = g_hash_table_new_full (g_direct_hash, g_direct_equal, NULL,
 									 theme_listener_free);
@@ -285,25 +223,10 @@ theme_manager_init (void)
 	if (!theme_manager_setup_listener_id)
 		theme_manager_setup_listener_id = theme_listener_register ("setup.apply", theme_manager_setup_apply_listener, NULL);
 
-	theme_policy_init ();
+	fe_set_auto_dark_mode_state (FALSE);
-	theme_policy_set_appearance_changed_callback (theme_manager_refresh_auto_mode);
-
-	settings = gtk_settings_get_default ();
-	if (settings && !settings_signals_connected)
-	{
-		g_signal_connect (settings, "notify::gtk-theme-name",
-				  G_CALLBACK (theme_manager_queue_auto_refresh), NULL);
-		g_signal_connect (settings, "notify::gtk-application-prefer-dark-theme",
-				  G_CALLBACK (theme_manager_queue_auto_refresh), NULL);
-		settings_signals_connected = TRUE;
-	}
-
-	fe_set_auto_dark_mode_state (prefs.hex_gui_dark_mode == ZOITECHAT_DARK_MODE_AUTO ?
-				     theme_policy_system_prefers_dark () : FALSE);
 	theme_application_apply_mode (prefs.hex_gui_dark_mode, NULL);
 	theme_gtk3_init ();
 	zoitechat_set_theme_post_apply_callback (theme_manager_handle_theme_applied);
-	theme_manager_auto_refresh_cache_prime ();
 }
 
 gboolean
@@ -590,9 +513,13 @@ theme_manager_apply_platform_window_theme (GtkWidget *window)
 
 	context = gtk_widget_get_style_context (window);
 	if (theme_gtk3_is_active ())
-		dark = theme_gtk3_active_variant () == THEME_GTK3_VARIANT_PREFER_DARK;
+	{
+		dark = prefs.hex_gui_gtk3_variant == THEME_GTK3_VARIANT_PREFER_DARK;
+		if (prefs.hex_gui_gtk3_variant == THEME_GTK3_VARIANT_FOLLOW_SYSTEM)
+			dark = theme_policy_system_prefers_dark ();
+	}
 	else
-		dark = theme_policy_is_dark_mode_active (prefs.hex_gui_dark_mode);
+		dark = theme_runtime_is_dark_active ();
 	if (context)
 	{
 		gtk_style_context_remove_class (context, "zoitechat-dark");
@@ -785,5 +712,4 @@ theme_manager_set_idle_add_func (ThemeManagerIdleAddFunc idle_add_func)
 {
 	theme_manager_idle_add_func = idle_add_func ? idle_add_func : g_idle_add;
 	theme_manager_auto_refresh_source = 0;
-	theme_manager_auto_refresh_cache_reset ();
 }

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

@@ -20,379 +20,14 @@
 
 #include "theme-policy.h"
 
-#include <string.h>
-
-#include <gtk/gtk.h>
-#include <gio/gio.h>
-
-#include "theme-gtk3.h"
 #include "../fe-gtk.h"
 #include "../../common/zoitechat.h"
 #include "../../common/zoitechatc.h"
 
-/* org.freedesktop.appearance color-scheme values (XDG desktop portal) */
-enum
-{
-	THEME_POLICY_COLOR_SCHEME_NO_PREFERENCE = 0,
-	THEME_POLICY_COLOR_SCHEME_PREFER_DARK = 1,
-	THEME_POLICY_COLOR_SCHEME_PREFER_LIGHT = 2
-};
-
-static ThemePolicyAppearanceChangedFunc policy_appearance_changed_cb;
-
-/* GtkSettings state captured before ZoiteChat writes to it, so that values
- * coming from the user's own GTK configuration are not mistaken for values
- * this application set. */
-static gboolean policy_initial_gtk_state_captured;
-static gboolean policy_initial_prefer_dark;
-static char *policy_initial_theme_name;
-
-#ifndef G_OS_WIN32
-static gboolean policy_portal_watch_started;
-static gboolean policy_portal_value_valid;
-static guint32 policy_portal_color_scheme;
-
-static gboolean policy_gsettings_checked;
-static GSettings *policy_gsettings_interface;
-#endif
-
-static void
-policy_capture_initial_gtk_state (void)
-{
-	GtkSettings *settings;
-
-	if (policy_initial_gtk_state_captured)
-		return;
-
-	settings = gtk_settings_get_default ();
-	if (!settings)
-		return;
-
-	g_object_get (settings,
-	              "gtk-application-prefer-dark-theme", &policy_initial_prefer_dark,
-	              "gtk-theme-name", &policy_initial_theme_name,
-	              NULL);
-	policy_initial_gtk_state_captured = TRUE;
-}
-
-static gboolean
-policy_theme_name_is_dark (const char *name)
-{
-	char *lower;
-	gboolean dark;
-
-	if (!name || !name[0])
-		return FALSE;
-
-	lower = g_ascii_strdown (name, -1);
-	dark = strstr (lower, "dark") != NULL || strstr (lower, "noir") != NULL ||
-	       strstr (lower, "black") != NULL;
-	g_free (lower);
-	return dark;
-}
-
-#ifndef G_OS_WIN32
-
-static void
-policy_notify_appearance_changed (void)
-{
-	if (policy_appearance_changed_cb)
-		policy_appearance_changed_cb ();
-}
-
-static gboolean
-policy_color_scheme_from_variant (GVariant *value, guint32 *color_scheme)
-{
-	GVariant *inner;
-	gboolean ok = FALSE;
-
-	if (!value || !color_scheme)
-		return FALSE;
-
-	inner = g_variant_ref (value);
-	while (g_variant_is_of_type (inner, G_VARIANT_TYPE_VARIANT))
-	{
-		GVariant *unwrapped = g_variant_get_variant (inner);
-
-		g_variant_unref (inner);
-		inner = unwrapped;
-	}
-
-	if (g_variant_is_of_type (inner, G_VARIANT_TYPE_UINT32))
-	{
-		*color_scheme = g_variant_get_uint32 (inner);
-		ok = TRUE;
-	}
-
-	g_variant_unref (inner);
-	return ok;
-}
-
-static void
-policy_portal_store_color_scheme (guint32 color_scheme)
-{
-	gboolean changed;
-
-	changed = !policy_portal_value_valid || policy_portal_color_scheme != color_scheme;
-	policy_portal_value_valid = TRUE;
-	policy_portal_color_scheme = color_scheme;
-
-	if (changed)
-		policy_notify_appearance_changed ();
-}
-
-static void
-policy_portal_setting_changed_cb (GDBusConnection *connection, const char *sender_name,
-                                  const char *object_path, const char *interface_name,
-                                  const char *signal_name, GVariant *parameters,
-                                  gpointer user_data)
-{
-	const char *setting_namespace = NULL;
-	const char *setting_key = NULL;
-	GVariant *value = NULL;
-	guint32 color_scheme = THEME_POLICY_COLOR_SCHEME_NO_PREFERENCE;
-
-	(void) connection;
-	(void) sender_name;
-	(void) object_path;
-	(void) interface_name;
-	(void) signal_name;
-	(void) user_data;
-
-	if (!parameters || !g_variant_is_of_type (parameters, G_VARIANT_TYPE ("(ssv)")))
-		return;
-
-	g_variant_get (parameters, "(&s&sv)", &setting_namespace, &setting_key, &value);
-	if (g_strcmp0 (setting_namespace, "org.freedesktop.appearance") == 0 &&
-	    g_strcmp0 (setting_key, "color-scheme") == 0 &&
-	    policy_color_scheme_from_variant (value, &color_scheme))
-		policy_portal_store_color_scheme (color_scheme);
-
-	if (value)
-		g_variant_unref (value);
-}
-
-static void
-policy_portal_read_ready_cb (GObject *source, GAsyncResult *result, gpointer user_data)
-{
-	GVariant *reply;
-	GVariant *value = NULL;
-	guint32 color_scheme = THEME_POLICY_COLOR_SCHEME_NO_PREFERENCE;
-
-	(void) user_data;
-
-	reply = g_dbus_connection_call_finish (G_DBUS_CONNECTION (source), result, NULL);
-	if (!reply)
-		return;
-
-	if (g_variant_is_of_type (reply, G_VARIANT_TYPE ("(v)")))
-	{
-		g_variant_get (reply, "(v)", &value);
-		if (policy_color_scheme_from_variant (value, &color_scheme))
-			policy_portal_store_color_scheme (color_scheme);
-		if (value)
-			g_variant_unref (value);
-	}
-
-	g_variant_unref (reply);
-}
-
-static void
-policy_portal_bus_ready_cb (GObject *source, GAsyncResult *result, gpointer user_data)
-{
-	GDBusConnection *connection;
-
-	(void) source;
-	(void) user_data;
-
-	connection = g_bus_get_finish (result, NULL);
-	if (!connection)
-		return;
-
-	g_dbus_connection_signal_subscribe (connection,
-	                                    "org.freedesktop.portal.Desktop",
-	                                    "org.freedesktop.portal.Settings",
-	                                    "SettingChanged",
-	                                    "/org/freedesktop/portal/desktop",
-	                                    NULL,
-	                                    G_DBUS_SIGNAL_FLAGS_NONE,
-	                                    policy_portal_setting_changed_cb,
-	                                    NULL,
-	                                    NULL);
-
-	g_dbus_connection_call (connection,
-	                        "org.freedesktop.portal.Desktop",
-	                        "/org/freedesktop/portal/desktop",
-	                        "org.freedesktop.portal.Settings",
-	                        "Read",
-	                        g_variant_new ("(ss)", "org.freedesktop.appearance", "color-scheme"),
-	                        G_VARIANT_TYPE ("(v)"),
-	                        G_DBUS_CALL_FLAGS_NONE,
-	                        3000,
-	                        NULL,
-	                        policy_portal_read_ready_cb,
-	                        NULL);
-	g_object_unref (connection);
-}
-
-static void
-policy_portal_watch_start (void)
-{
-	if (policy_portal_watch_started)
-		return;
-
-	policy_portal_watch_started = TRUE;
-	g_bus_get (G_BUS_TYPE_SESSION, NULL, policy_portal_bus_ready_cb, NULL);
-}
-
-static void
-policy_gsettings_changed_cb (GSettings *settings, const char *key, gpointer user_data)
-{
-	(void) settings;
-	(void) key;
-	(void) user_data;
-
-	policy_notify_appearance_changed ();
-}
-
-static GSettings *
-policy_get_interface_gsettings (void)
-{
-	GSettingsSchemaSource *source;
-	GSettingsSchema *schema;
-	gboolean has_key;
-
-	if (policy_gsettings_checked)
-		return policy_gsettings_interface;
-
-	policy_gsettings_checked = TRUE;
-
-	source = g_settings_schema_source_get_default ();
-	if (!source)
-		return NULL;
-
-	schema = g_settings_schema_source_lookup (source, "org.gnome.desktop.interface", TRUE);
-	if (!schema)
-		return NULL;
-
-	/* Requires GLib 2.40, always satisfied by the GTK 3.22 dependency. */
-	G_GNUC_BEGIN_IGNORE_DEPRECATIONS
-	has_key = g_settings_schema_has_key (schema, "color-scheme");
-	G_GNUC_END_IGNORE_DEPRECATIONS
-	g_settings_schema_unref (schema);
-	if (!has_key)
-		return NULL;
-
-	policy_gsettings_interface = g_settings_new ("org.gnome.desktop.interface");
-	g_signal_connect (policy_gsettings_interface, "changed::color-scheme",
-	                  G_CALLBACK (policy_gsettings_changed_cb), NULL);
-	return policy_gsettings_interface;
-}
-
-static gboolean
-policy_gsettings_color_scheme (gboolean *prefers_dark)
-{
-	GSettings *settings;
-	char *scheme;
-	gboolean explicit_preference = FALSE;
-
-	settings = policy_get_interface_gsettings ();
-	if (!settings)
-		return FALSE;
-
-	scheme = g_settings_get_string (settings, "color-scheme");
-	if (g_strcmp0 (scheme, "prefer-dark") == 0)
-	{
-		*prefers_dark = TRUE;
-		explicit_preference = TRUE;
-	}
-	else if (g_strcmp0 (scheme, "prefer-light") == 0)
-	{
-		*prefers_dark = FALSE;
-		explicit_preference = TRUE;
-	}
-	g_free (scheme);
-	return explicit_preference;
-}
-
-static gboolean
-policy_gtk_heuristic_prefers_dark (void)
-{
-	GtkSettings *settings;
-	char *theme_name = NULL;
-	gboolean dark;
-
-	policy_capture_initial_gtk_state ();
-
-	/* gtk-application-prefer-dark-theme is overwritten by ZoiteChat at
-	 * runtime, so only its captured startup value (settings.ini, XSETTINGS)
-	 * reflects the user's own configuration. */
-	if (policy_initial_gtk_state_captured && policy_initial_prefer_dark)
-		return TRUE;
-
-	if (theme_gtk3_is_active ())
-	{
-		/* gtk-theme-name currently holds the in-app theme override;
-		 * fall back to the captured system theme name. */
-		theme_name = g_strdup (policy_initial_theme_name);
-	}
-	else
-	{
-		settings = gtk_settings_get_default ();
-		if (settings)
-			g_object_get (settings, "gtk-theme-name", &theme_name, NULL);
-	}
-
-	dark = policy_theme_name_is_dark (theme_name);
-	g_free (theme_name);
-	return dark;
-}
-
-#endif /* !G_OS_WIN32 */
-
-void
-theme_policy_init (void)
-{
-	policy_capture_initial_gtk_state ();
-#ifndef G_OS_WIN32
-	policy_portal_watch_start ();
-#endif
-}
-
-void
-theme_policy_set_appearance_changed_callback (ThemePolicyAppearanceChangedFunc callback)
-{
-	policy_appearance_changed_cb = callback;
-}
-
 gboolean
 theme_policy_system_prefers_dark (void)
 {
-#ifdef G_OS_WIN32
-	gboolean prefer_dark = FALSE;
-
-	policy_capture_initial_gtk_state ();
-
-	if (fe_win32_high_contrast_is_enabled ())
 	return FALSE;
-	if (fe_win32_try_get_system_dark (&prefer_dark))
-		return prefer_dark;
-	return policy_theme_name_is_dark (policy_initial_theme_name);
-#else
-	gboolean prefers_dark = FALSE;
-
-	policy_capture_initial_gtk_state ();
-	policy_portal_watch_start ();
-
-	if (policy_portal_value_valid &&
-	    policy_portal_color_scheme != THEME_POLICY_COLOR_SCHEME_NO_PREFERENCE)
-		return policy_portal_color_scheme == THEME_POLICY_COLOR_SCHEME_PREFER_DARK;
-
-	if (policy_gsettings_color_scheme (&prefers_dark))
-		return prefers_dark;
-
-	return policy_gtk_heuristic_prefers_dark ();
-#endif
 }
 
 gboolean

src/fe-gtk/theme/theme-policy.h

@@ -23,10 +23,6 @@
 
 #include "../fe-gtk.h"
 
-typedef void (*ThemePolicyAppearanceChangedFunc) (void);
-
-void theme_policy_init (void);
-void theme_policy_set_appearance_changed_callback (ThemePolicyAppearanceChangedFunc callback);
 gboolean theme_policy_system_prefers_dark (void);
 gboolean theme_policy_is_dark_mode_active (unsigned int mode);
 gboolean theme_policy_is_app_dark_mode_active (void);

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

@@ -101,16 +101,8 @@ typedef struct
         gboolean changed;
         gboolean snapshot_valid[THEME_TOKEN_COUNT];
         gboolean staged_valid[THEME_TOKEN_COUNT];
-        /* Only explicitly edited tokens are written back to the runtime
-         * palette; untouched tokens keep following the GTK3 theme. */
-        gboolean staged_dirty[THEME_TOKEN_COUNT];
         GdkRGBA snapshot[THEME_TOKEN_COUNT];
         GdkRGBA staged[THEME_TOKEN_COUNT];
-        /* the GTK3 theme selection is previewed live from the combo, so
-         * remember what was active when the dialog opened in order to
-         * restore it when the dialog is cancelled */
-        char gtk3_theme_snapshot[sizeof ((struct zoitechatprefs *) 0)->hex_gui_gtk3_theme];
-        int gtk3_variant_snapshot;
 } theme_preferences_stage_state;
 
 static theme_preferences_stage_state theme_preferences_stage;
@@ -155,8 +147,7 @@ theme_preferences_stage_sync_runtime_to_snapshot (void)
 
         for (token = THEME_TOKEN_MIRC_0; token < THEME_TOKEN_COUNT; token++)
         {
-                if (theme_preferences_stage.snapshot_valid[token] &&
+                if (theme_preferences_stage.snapshot_valid[token])
-                    theme_preferences_stage.staged_dirty[token])
                         theme_manager_set_token_color (ZOITECHAT_DARK_MODE_LIGHT, token,
                                                        &theme_preferences_stage.snapshot[token], NULL);
         }
@@ -169,8 +160,7 @@ theme_preferences_stage_sync_runtime_to_staged (void)
 
         for (token = THEME_TOKEN_MIRC_0; token < THEME_TOKEN_COUNT; token++)
         {
-                if (theme_preferences_stage.staged_valid[token] &&
+                if (theme_preferences_stage.staged_valid[token])
-                    theme_preferences_stage.staged_dirty[token])
                         theme_manager_set_token_color (ZOITECHAT_DARK_MODE_LIGHT, token,
                                                        &theme_preferences_stage.staged[token], NULL);
         }
@@ -189,7 +179,6 @@ theme_preferences_staged_set_color (ThemeSemanticToken token, const GdkRGBA *rgb
         {
                 theme_preferences_stage.staged[token] = *rgba;
                 theme_preferences_stage.staged_valid[token] = TRUE;
-                theme_preferences_stage.staged_dirty[token] = TRUE;
                 theme_preferences_stage_recompute_changed ();
                 if (color_change_flag)
                         *color_change_flag = theme_preferences_stage.changed;
@@ -209,11 +198,6 @@ theme_preferences_stage_begin (void)
         memset (&theme_preferences_stage, 0, sizeof (theme_preferences_stage));
         theme_preferences_stage.active = TRUE;
 
-        g_strlcpy (theme_preferences_stage.gtk3_theme_snapshot,
-                   prefs.hex_gui_gtk3_theme,
-                   sizeof (theme_preferences_stage.gtk3_theme_snapshot));
-        theme_preferences_stage.gtk3_variant_snapshot = prefs.hex_gui_gtk3_variant;
-
         for (token = THEME_TOKEN_MIRC_0; token < THEME_TOKEN_COUNT; token++)
         {
                 GdkRGBA rgba;
@@ -247,36 +231,12 @@ theme_preferences_stage_commit (void)
         memset (&theme_preferences_stage, 0, sizeof (theme_preferences_stage));
 }
 
-/* undo a live GTK3 theme preview: put the selection that was active when
- * the dialog opened back and re-apply it */
-static void
-theme_preferences_stage_restore_gtk3 (void)
-{
-        if (g_strcmp0 (prefs.hex_gui_gtk3_theme, theme_preferences_stage.gtk3_theme_snapshot) == 0 &&
-            prefs.hex_gui_gtk3_variant == theme_preferences_stage.gtk3_variant_snapshot)
-                return;
-
-        g_strlcpy (prefs.hex_gui_gtk3_theme,
-                   theme_preferences_stage.gtk3_theme_snapshot,
-                   sizeof (prefs.hex_gui_gtk3_theme));
-        prefs.hex_gui_gtk3_variant = theme_preferences_stage.gtk3_variant_snapshot;
-
-        theme_gtk3_apply_current (NULL);
-        theme_manager_dispatch_changed (THEME_CHANGED_REASON_THEME_PACK |
-                                        THEME_CHANGED_REASON_PALETTE |
-                                        THEME_CHANGED_REASON_WIDGET_STYLE |
-                                        THEME_CHANGED_REASON_USERLIST |
-                                        THEME_CHANGED_REASON_MODE);
-}
-
 void
 theme_preferences_stage_discard (void)
 {
         if (!theme_preferences_stage.active)
                 return;
 
-        /* restore the GTK3 theme first so the color snapshot below wins */
-        theme_preferences_stage_restore_gtk3 ();
         theme_preferences_stage_sync_runtime_to_snapshot ();
         memset (&theme_preferences_stage, 0, sizeof (theme_preferences_stage));
 }
@@ -881,10 +841,6 @@ theme_preferences_manager_dialog_response_cb (GtkDialog *dialog, gint response_i
                         ThemeSemanticToken token;
                         ThemeWidgetStyleValues style_values;
 
-                        /* The runtime palette was just reset; drop any
-                         * pending edits so the reset is not re-pinned. */
-                        memset (theme_preferences_stage.staged_dirty, 0,
-                                sizeof (theme_preferences_stage.staged_dirty));
                         for (token = THEME_TOKEN_MIRC_0; token < THEME_TOKEN_COUNT; token++)
                         {
                                 GdkRGBA rgba;
@@ -899,6 +855,7 @@ theme_preferences_manager_dialog_response_cb (GtkDialog *dialog, gint response_i
                         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;
@@ -1577,25 +1534,6 @@ theme_preferences_gtk3_sync_remove_state (theme_preferences_ui *ui)
         gtk_widget_set_sensitive (ui->gtk3_remove, source == ZOITECHAT_GTK3_THEME_SOURCE_USER);
 }
 
-static void
-theme_preferences_stage_rebase_color (ThemeSemanticToken token, const GdkRGBA *rgba)
-{
-	if (token < 0 || token >= THEME_TOKEN_COUNT || !rgba)
-		return;
-	if (!theme_preferences_stage.active)
-		return;
-
-	/* The theme changed underneath the dialog: shift the baseline so the
-	 * new theme-provided value is neither treated as a user edit nor
-	 * pinned into the runtime palette. */
-	theme_preferences_stage.snapshot[token] = *rgba;
-	theme_preferences_stage.staged[token] = *rgba;
-	theme_preferences_stage.snapshot_valid[token] = TRUE;
-	theme_preferences_stage.staged_valid[token] = TRUE;
-	theme_preferences_stage.staged_dirty[token] = FALSE;
-	theme_preferences_stage_recompute_changed ();
-}
-
 static void
 theme_preferences_gtk3_sync_runtime_palette (theme_preferences_ui *ui)
 {
@@ -1611,16 +1549,22 @@ theme_preferences_gtk3_sync_runtime_palette (theme_preferences_ui *ui)
 
 	theme_get_widget_style_values_for_widget (style_source, &style_values);
 
-	/* Leave the mapped tokens unpinned so the user list, channel tree and
+	theme_preferences_staged_set_color (THEME_TOKEN_TEXT_FOREGROUND,
-	 * input box keep resolving their colors from whatever GTK3 theme is
+	                                    &style_values.foreground,
-	 * active (system or in-app selected), and only refresh what the
+	                                    NULL,
-	 * dialog shows.  Pinning theme-derived values here freezes them into
+	                                    TRUE);
-	 * the runtime palette (and colors.conf), which left those widgets
+	theme_preferences_staged_set_color (THEME_TOKEN_TEXT_BACKGROUND,
-	 * with stale colors until the next restart. */
+	                                    &style_values.background,
-	theme_preferences_stage_rebase_color (THEME_TOKEN_TEXT_FOREGROUND, &style_values.foreground);
+	                                    NULL,
-	theme_preferences_stage_rebase_color (THEME_TOKEN_TEXT_BACKGROUND, &style_values.background);
+	                                    TRUE);
-	theme_preferences_stage_rebase_color (THEME_TOKEN_SELECTION_FOREGROUND, &style_values.selection_foreground);
+	theme_preferences_staged_set_color (THEME_TOKEN_SELECTION_FOREGROUND,
-	theme_preferences_stage_rebase_color (THEME_TOKEN_SELECTION_BACKGROUND, &style_values.selection_background);
+	                                    &style_values.selection_foreground,
+	                                    NULL,
+	                                    TRUE);
+	theme_preferences_staged_set_color (THEME_TOKEN_SELECTION_BACKGROUND,
+	                                    &style_values.selection_background,
+	                                    NULL,
+	                                    TRUE);
 }
 
 static gboolean

src/fe-gtk/userlistgui.c

@@ -796,12 +796,16 @@ userlist_store_color (GtkListStore *store, GtkTreeIter *iter, ThemeSemanticToken
 	}
 }
 
-static void
+GtkListStore *
-userlist_model_set_sort (GtkListStore *store, session *sess)
+userlist_create_model (session *sess)
 {
+	GtkListStore *store;
 	GtkTreeIterCompareFunc cmp_func;
 	GtkSortType sort_type;
 
+	store = gtk_list_store_new (6, GDK_TYPE_PIXBUF, G_TYPE_STRING, G_TYPE_STRING,
+										G_TYPE_STRING, G_TYPE_POINTER, THEME_GTK_COLOR_TYPE);
+
 	switch (prefs.hex_gui_ulist_sort)
 	{
 	case 0:
@@ -821,77 +825,18 @@ userlist_model_set_sort (GtkListStore *store, session *sess)
 		sort_type = GTK_SORT_DESCENDING;
 		break;
 	default:
-		/* No sorting; rows keep their current order and new rows
+		/* No sorting */
-		   go wherever they are inserted */
-		gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE(store),
-							GTK_TREE_SORTABLE_UNSORTED_SORT_COLUMN_ID, GTK_SORT_ASCENDING);
 		gtk_tree_sortable_set_default_sort_func (GTK_TREE_SORTABLE(store), NULL, NULL, NULL);
-		return;
+		return store;
 	}
 
 	gtk_tree_sortable_set_default_sort_func (GTK_TREE_SORTABLE(store), cmp_func, sess, NULL);
 	gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE(store),
 						GTK_TREE_SORTABLE_DEFAULT_SORT_COLUMN_ID, sort_type);
-}
-
-GtkListStore *
-userlist_create_model (session *sess)
-{
-	GtkListStore *store;
-
-	store = gtk_list_store_new (6, GDK_TYPE_PIXBUF, G_TYPE_STRING, G_TYPE_STRING,
-										G_TYPE_STRING, G_TYPE_POINTER, THEME_GTK_COLOR_TYPE);
-	userlist_model_set_sort (store, sess);
 
 	return store;
 }
 
-void
-userlist_apply_sort (session *sess)
-{
-	if (!sess->res || !sess->res->user_model)
-		return;
-
-	userlist_model_set_sort (sess->res->user_model, sess);
-}
-
-/* remove and re-insert every row, so per-row content that depends on
- * preferences (mode icons vs. text prefixes, hostnames) is rebuilt */
-void
-userlist_refill (session *sess)
-{
-	GtkTreeModel *model;
-	GtkTreeIter iter;
-	GSList *users = NULL;
-	GSList *it;
-	struct User *user;
-
-	if (!sess->res || !sess->res->user_model)
-		return;
-
-	model = GTK_TREE_MODEL (sess->res->user_model);
-	if (gtk_tree_model_get_iter_first (model, &iter))
-	{
-		do
-		{
-			gtk_tree_model_get (model, &iter, COL_USER, &user, -1);
-			if (user)
-				users = g_slist_prepend (users, user);
-		}
-		while (gtk_tree_model_iter_next (model, &iter));
-	}
-
-	fe_userlist_clear (sess);
-
-	/* fe_userlist_insert() prepends, so inserting in reverse display
-	   order restores the on-screen order for unsorted lists; sorted
-	   models position the rows themselves */
-	for (it = users; it; it = it->next)
-		fe_userlist_insert (sess, it->data, FALSE);
-
-	g_slist_free (users);
-}
-
 static void
 userlist_add_columns (GtkTreeView * treeview)
 {
@@ -961,23 +906,6 @@ userlist_add_columns (GtkTreeView * treeview)
 	}
 }
 
-void
-userlist_rebuild_columns (GtkWidget *treeview)
-{
-	GList *columns;
-	GList *it;
-
-	if (!treeview)
-		return;
-
-	columns = gtk_tree_view_get_columns (GTK_TREE_VIEW (treeview));
-	for (it = columns; it; it = it->next)
-		gtk_tree_view_remove_column (GTK_TREE_VIEW (treeview), it->data);
-	g_list_free (columns);
-
-	userlist_add_columns (GTK_TREE_VIEW (treeview));
-}
-
 static gint
 userlist_click_cb (GtkWidget *widget, GdkEventButton *event, gpointer userdata)
 {

src/fe-gtk/userlistgui.h

@@ -24,9 +24,6 @@ void userlist_set_value (GtkWidget *treeview, gfloat val);
 gfloat userlist_get_value (GtkWidget *treeview);
 GtkWidget *userlist_create (GtkWidget *box);
 GtkListStore *userlist_create_model (session *sess);
-void userlist_apply_sort (session *sess);
-void userlist_refill (session *sess);
-void userlist_rebuild_columns (GtkWidget *treeview);
 void userlist_show (session *sess);
 void userlist_select (session *sess, char *name);
 void fe_userlist_set_typing (session *sess, const char *nick, const char *state);

src/fe-text/fe-text.c

@@ -857,10 +857,6 @@ fe_get_str (char *prompt, char *def, void *callback, void *ud)
 {
 }
 void
-fe_get_password (char *prompt, void *callback, void *ud)
-{
-}
-void
 fe_get_int (char *prompt, int def, void *callback, void *ud)
 {
 }

win32/config.h.tt

@@ -9,6 +9,7 @@
 #define PACKAGE_VERSION "<#= [string]::Join('.', $versionParts) #>"
 #define ZOITECHATLIBDIR ".\\plugins"
 #define ZOITECHATSHAREDIR "."
+#define ZOITECHATDOCDIR "offline-docs"
 #define OLD_PERL
 #define GETTEXT_PACKAGE "zoitechat"
 #define PACKAGE_TARNAME "zoitechat-<#= [string]::Join('.', $versionParts) #>"

win32/installer/zoitechat.iss.tt

@@ -49,6 +49,7 @@ Name: "icons"; Description: "Create Shortcuts"; Types: custom; Flags: disablenou
 Name: "icons\desktopicon"; Description: "Create Desktop Shortcut"; Types: custom; Flags: disablenouninstallwarning
 Name: "icons\quicklaunchicon"; Description: "Create Quick Launch Shortcut"; Types: custom; Flags: disablenouninstallwarning
 Name: "translations"; Description: "Translations"; Types: normal custom; Flags: disablenouninstallwarning
+Name: "docs"; Description: "Offline Documentation"; Types: normal custom; Flags: disablenouninstallwarning
 Name: "spell"; Description: "Spelling Dictionaries"; Types: custom; Flags: disablenouninstallwarning
 Name: "plugins"; Description: "Plugins"; Types: custom; Flags: disablenouninstallwarning
 Name: "plugins\checksum"; Description: "Checksum"; Types: custom; Flags: disablenouninstallwarning
@@ -109,6 +110,9 @@ Source: "share\themes\MS-Windows\*"; DestDir: "{app}\share\themes\MS-Windows"; F
 Source: "share\glib-2.0\schemas\*"; DestDir: "{app}\share\glib-2.0\schemas"; Flags: ignoreversion createallsubdirs recursesubdirs skipifsourcedoesntexist; Components: libs
 Source: "share\icons\hicolor\*"; DestDir: "{app}\share\icons\hicolor"; Flags: ignoreversion createallsubdirs recursesubdirs skipifsourcedoesntexist; Components: libs
 Source: "share\locale\*"; DestDir: "{app}\share\locale"; Flags: ignoreversion createallsubdirs recursesubdirs; Components: translations
+; Staged at build time by data\misc\build_offline_docs.py; absent when the
+; docs could not be built, in which case Help->Contents opens the website
+Source: "offline-docs\*"; DestDir: "{app}\offline-docs"; Flags: ignoreversion createallsubdirs recursesubdirs skipifsourcedoesntexist; Components: docs
 Source: "etc\fonts\*"; DestDir: "{app}\etc\fonts"; Flags: ignoreversion createallsubdirs recursesubdirs; Components: libs
 
 Source: "atk-1.0-0.dll"; DestDir: "{app}"; Flags: ignoreversion; Components: libs