Skip to content

Conversation

@vtyrtov
Copy link
Contributor

@vtyrtov vtyrtov commented Nov 8, 2025

Issue #19806

Summary

Keep the original Wayland behavior from master (GtkHeaderBar / client-side decorations). Add a single opt-in flag so GNOME Shell extensions (e.g. Unite) can hide the titlebar cleanly and reclaim space.

Behavior

Default (Wayland): GtkHeaderBar shown, unchanged from master.
Opt-in compatibility mode:
Enable via DARKTABLE_HIDE_TITLEBAR_ON_MAXIMIZE=1 (or ui/hide_titlebar_on_maximize=true)
Effect: no GtkHeaderBar, server-side decorations used, gtk_window_set_hide_titlebar_when_maximized(TRUE). Allows WM/extension to hide titlebar without a gap.

Implementation

Added conditional logic in [gtk.c]

Testing

Default: header bar visible as before.
With DARKTABLE_HIDE_TITLEBAR_ON_MAXIMIZE=1: Unite hides titlebar; window resizes (no top gap).

@vtyrtov vtyrtov changed the title Wayland: add DARKTABLE_HIDE_TITLEBAR_ON_MAXIMIZE to disable HeaderBar for Unite compatibility Wayland: add DARKTABLE_HIDE_TITLEBAR_ON_MAXIMIZE to disable TitleBar for Unite compatibility Nov 9, 2025
@vtyrtov vtyrtov closed this Nov 11, 2025
@vtyrtov vtyrtov reopened this Nov 24, 2025
…ed hint

This commit adds multiple options for controlling window decorations on Wayland:

1. Prefer server-side decorations (SSD) over client-side decorations (CSD)
2. Add hide-titlebar-when-maximized hint for better compatibility with
   GNOME extensions like Unite
3. Add environment variable DARKTABLE_HIDE_TITLEBAR_ON_MAXIMIZE to opt-out
   of HeaderBar usage when maximized (for Unite compatibility)
4. Add darktablerc option 'hide_titlebar_on_maximize' for persistent configuration

These changes improve window behavior on Wayland, especially with tiling
window managers and GNOME Shell extensions.
@vtyrtov vtyrtov force-pushed the pr/wayland-titlebar-hint-clean branch from 39f9d6a to 7d23b6b Compare November 24, 2025 10:20
@vtyrtov vtyrtov marked this pull request as ready for review November 24, 2025 10:21
@victoryforce victoryforce self-requested a review November 25, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant