helix: use theme option

[0xda157]

as discussed in #481

Things done

• Tested locally
• Tested in testbed
• Commit message follows commit convention
• Fits style guide
• Respects license of any existing code used

Notify maintainers

@danth (based on blame)

[0xda157]

I had to convert the base16-helix template into nix

[Flameopathic]

I think that it is better that we rely on outside templates for things like this because it, in general, keeps the templates more up-to-date than a Stylix contributor would. Consider opening a PR upstream for allowing this option to take strings and/or paths as well so that we can continue using base16-helix.

[trueNAHO]

I think that it is better that we rely on outside templates for things like this because it, in general, keeps the templates more up-to-date than a Stylix contributor would. Consider opening a PR upstream for allowing this option to take strings and/or paths as well so that we can continue using base16-helix.

Yes, ideally .settings or .theme options would be upstreamed to platforms like Home Manager:

This issue covers a real tangible Stylix limitation that is planned to be resolved in the future.

is our plan of resolving this the same as in #481 as well, which is upstreaming options?

AFAIK, yes.

-- #704 (comment)

Since this is a huge undertaking there has not really been any progress on this. AFAIK nix-community/home-manager#6235 (comment) is the only time this has been upstreamed.

[trueNAHO]

LGTM. Thanks for upstreaming this.

This PR will be merged in #1146.