matugen icon indicating copy to clipboard operation
matugen copied to clipboard

`{{mode}}` is upper case

Open Curve opened this issue 2 months ago • 4 comments

In the themes repo, it is suggested to use this post_hook for GTK3:

gsettings set org.gnome.desktop.interface gtk-theme ''; gsettings set org.gnome.desktop.interface gtk-theme adw-gtk3-{{mode}}

However, mode is replaced with Dark for dark, which results in the theme update failing, as GTK expects adw-gtk3-dark (notice the lower case d).

I tried to append | to_lower in the template but that results in matugen failing to parse the hook.

Curve avatar Oct 16 '25 14:10 Curve

Solved with:

post_hook = 'gsettings set org.gnome.desktop.interface gtk-theme ""; gsettings set org.gnome.desktop.interface gtk-theme adw-gtk3{{mode | lower_case | replace: "light", "" | replace: "d", "-d"}}'

maybe this should be updated in the themes repo?

Curve avatar Oct 16 '25 15:10 Curve

Another method:

[templates.gtk3]
input_path = './templates/gtk/colors.css'
output_path = '~/.config/gtk-3.0/colors.css'
post_hook = 'gsettings set org.gnome.desktop.interface gtk-theme ""; echo {{mode }}; gsettings set org.gnome.desktop.interface gtk-theme {{ mode | replace: "Dark", "adw-gtk3-dark" | replace: "Light", "adw-gtk3" }} && gsettings set org.gnome.desktop.interface color-scheme "{{ mode | replace: "Dark", "prefer-dark" | replace: "Light", "default" }}"'

[templates.gtk4]
input_path = './templates/gtk/colors.css'
output_path = '~/.config/gtk-4.0/colors.css'
post_hook = 'gsettings set org.gnome.desktop.interface gtk-theme ""; echo {{mode }}; gsettings set org.gnome.desktop.interface gtk-theme {{ mode | replace: "Dark", "adw-gtk3-dark" | replace: "Light", "adw-gtk3" }} && gsettings set org.gnome.desktop.interface color-scheme "{{ mode | replace: "Dark", "prefer-dark" | replace: "Light", "default" }}"'

enessmr avatar Oct 17 '25 13:10 enessmr

Should mode be lowercase by default? I could change that.

InioX avatar Nov 16 '25 21:11 InioX

Should mode be lowercase by default? I could change that.

That would be nice, I think there's few cases where one would need it with the first letter capitalized

Curve avatar Nov 16 '25 21:11 Curve

Should be fixed now

InioX avatar Nov 23 '25 08:11 InioX