Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GTK themes are not applied correctly #157

Closed
nikololiahim opened this issue May 2, 2024 · 4 comments
Closed

GTK themes are not applied correctly #157

nikololiahim opened this issue May 2, 2024 · 4 comments

Comments

@nikololiahim
Copy link

I have the following home-manager GTK config:

  catppuccin.flavour = "mocha";

  ...

  gtk = {
    enable = true;
    catppuccin = {
      enable = true;
      size = "compact";
      accent = "pink";
      tweaks = [ "rimless" ];
      cursor = {
        enable = true;
        accent = "dark";
      };
    };
  };

When I switch to this config using home-manager switch --flake, all the GTK apps have the white background. I have tried different revisions of catppuccin/nix flake and figured out that 1fbdfdacf96c14449aea52edba895e5ab419dd13 seems to be the culprit.

Example GTK app before 1fbdfdac:
image

After:
image

@isabelroses
Copy link
Member

The change to "Dark" -> "dark" and etc was made in regards to the upstream catppuccin/gtk. But nixos/nixpkgs still has not received these changes the maintainer has been contacted and hopefully this will be fixed soon.

@nikololiahim
Copy link
Author

I see, thank you! Then a temporary workaround would be to just pin the flake's git
rev and switch back to master when the GTK changes hit the unstable nixpkgs branch.

@isabelroses
Copy link
Member

The situation is a lot more complex then we first though. catppuccin/gtk uses a pinned commit submodule but nixos/nixpkgs uses a colloid release to build the catppuccin theme so even if we update catppuccin/gtk on nixpkgs the fix won't work so we are just going to roll back the change.

@getchoo
Copy link
Member

getchoo commented May 13, 2024

based on the comments in #147 and #159 being merged, this should be fixed. feel free to re-open if there are any more issues :)

@getchoo getchoo closed this as completed May 13, 2024
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

No branches or pull requests

3 participants