Skip to content

gnome: install User Themes extension#1306

Merged
danth merged 1 commit intomasterfrom
gnome-extension-install
May 18, 2025
Merged

gnome: install User Themes extension#1306
danth merged 1 commit intomasterfrom
gnome-extension-install

Conversation

@danth
Copy link
Copy Markdown
Member

@danth danth commented May 18, 2025

It was always intended that Stylix would install this extension itself, as the Home Manager module's functionality relies on it.

I'm not sure whether this code has always been missing, or if it was accidentally removed at some point, as I only noticed when #1305 started happening and I went to check whether the extension was enabled.

Note that the extension is less than 100 KiB and isn't executed unless GNOME is running (it's not on $PATH or anything), so although installing this for all users is not ideal, it shouldn't cause any major problems.

Things done

Notify maintainers

It was always intended that Stylix would install this extension itself,
as the Home Manager module's functionality relies on it.
@stylix-automation stylix-automation bot added the topic: home-manager Home Manager target label May 18, 2025
@danth danth enabled auto-merge (squash) May 18, 2025 12:52
Copy link
Copy Markdown
Member

@trueNAHO trueNAHO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was always intended that Stylix would install this extension itself, as the Home Manager module's functionality relies on it.

I'm not sure whether this code has always been missing, or if it was accidentally removed at some point, as I only noticed when #1305 started happening and I went to check whether the extension was enabled.

LGTM.

Note that the extension is less than 100 KiB and isn't executed unless GNOME is running (it's not on $PATH or anything), so although installing this for all users is not ideal, it shouldn't cause any major problems.

I assume some this would not be our first module to include potentially unnecessary dependencies. I suggest leaving these "performance" cleanups for the future.

@danth danth merged commit 2eaa338 into master May 18, 2025
7 checks passed
@danth danth deleted the gnome-extension-install branch May 18, 2025 14:26
@stylix-automation
Copy link
Copy Markdown
Contributor

Backport failed for release-24.11, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-24.11
git worktree add -d .worktree/backport-1306-to-release-24.11 origin/release-24.11
cd .worktree/backport-1306-to-release-24.11
git switch --create backport-1306-to-release-24.11
git cherry-pick -x 2eaa338eb879b8432f7e252d6ab8725ada98f52d

@alyraffauf
Copy link
Copy Markdown

FYI, this is causing build errors on nix-darwin with the hm module. pkgs.gnomeExtensions.user-themes is not available on darwin.

@danth
Copy link
Copy Markdown
Member Author

danth commented May 20, 2025

We should make the GNOME module only enabled by default on Linux

@0xda157
Copy link
Copy Markdown
Contributor

0xda157 commented May 20, 2025

We should make the GNOME module only enabled by default on Linux

do we ever want GNOME on darwin? if we change the default con that causes issues with #1244, if we add isLinux to config = mkIf then we won't have that issue with the docs.

@0xda157 0xda157 mentioned this pull request May 20, 2025
5 tasks
@danth
Copy link
Copy Markdown
Member Author

danth commented May 20, 2025

I doubt the target is useful there as it only themes GNOME Shell (so not anything you would actually have installed on MacOS)

MrSom3body pushed a commit to MrSom3body-contrib/stylix that referenced this pull request May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: home-manager Home Manager target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants