Remove 'variants' #601
Remove 'variants' #601
Conversation
In the sources there are 2 'variants' defined for fonts, and 3 for brightness (light/dark/darker). Do we plan to expose those at some point, or would it be a simpler to get rid of them?
|
See #508 for fonts; the choice hadn't been made yet. For the time being, since the goal is to self-host the fonts (thanks) just prepare the files for the variant in use, and document any non-straightforward thing you had to do to get self-hosting done. If we end up going over fonts choice (as we should have beforehand) hopefully we (me) can just pick the appropriate files. Keep the variant-B styles around, if only so we can remember which fonts goes where. Since it's not compiled down into the CSS output it's free except in maintainer cognitive load (and then it's basically not an issue). |
|
For any other kind of variants: don't remove any. (Though I see it's only been done for the navbar.) They are not present in the compiled CSS (if they are it should be fixed). This is something you couldn't have known as it came from internal talks about implementing this, but we might end up using variants under some circumstances. An example might be the Mobile NixOS website would use the same foundation for the site by importing the nixos homepage repository, but configure the darker menu rather than the lighter menu. Same for the search site, it's not totally decided what is going to happen exactly, but it might be using variants that are not in use on the main site. |
Yeah, saw that (I'm partial to the 'Source Sans Pro + Open Sans' option myself) - still we don't really need variants for that, we can just change it when we make the decision (possibly prototyping it in a branch).
Yeah, I figured as much - I didn't know where to bring it up and a PR seemed like a reasonable place to do it ;)
I think maintainer/contributor cognitive load is important :) - but if we see a potential use for them in the near(ish) future I'll close this PR 👍 |
|
And for completeness' sake, this was done that way during development to allow other contributors to trivially test the two proposed font stacks. |
In the sources there are 2 'variants' defined for fonts, and 3 for brightness
(light/dark/darker).
Do we plan to expose those at some point, or would it be a simpler to
get rid of them?
(the background is I'd like to look into self-hosting the fonts, but right now
it's not clear whether it makes sense to port all fonts or just the ones that
are used in 'variant A')