nixos-homepage icon indicating copy to clipboard operation
nixos-homepage copied to clipboard

[Techstack-beta] Nixpkgs and NixOS manuals regressed to default styles

Open samueldr opened this issue 1 year ago • 10 comments

This also applies to the pills.

image

samueldr avatar Mar 02 '24 00:03 samueldr

Deciding on how to embed the docs should, as the rest of the documentation, be a task of the docs team. But maybe @garbas kan tell you more about that.

Closing this for now, since it is intentional

thilobillerbeck avatar Mar 02 '24 01:03 thilobillerbeck

I'm sorry to dump this here, but this is not aimed at you, @thilobillerbeck.

This again shows that the execution of handling the technicalities of building the website shouldn't be handled by the marketing team. The marketing team should handle marketing. (And the marketing team shouldn't handle documentation. Which was a problem when I was still involved.)

Similarly, the documentation team should author documentation. Not deal with having the form of the documentation be coherent with the site.

This was a working feature.

This is a regression.

samueldr avatar Mar 03 '24 03:03 samueldr

Should this issue be open? The issue is still present on the website.

asymmetric avatar Mar 13 '24 06:03 asymmetric

The unthemed pills and manual is intended behavior. They will be also.managed by the docs team (hopefully soon) where theme can be applied.

garbas avatar Mar 13 '24 07:03 garbas

As an active docs team member this is the first time I hear about that..

Ping @NixOS/documentation-team, did anybody get informed about this?

infinisil avatar Mar 13 '24 08:03 infinisil

At some point we agreed to move the manuals to nix.dev, yes. I don't have the source right now.

I still intend to make it happen one way or another. I started in October and didn't manage to finish yet because of conflicting priorities due to too many responsibilities.

IMO it's not great to degrade appearance, but we clearly have more pressing issues than style sheets.

fricklerhandwerk avatar Mar 13 '24 11:03 fricklerhandwerk

...Huh. I'd noticed the "default" stylesheet but assumed it was just a new stylesheet. The reduced whitespace, particularly around the table of contents, makes it much easier to parse. I thought it was an improvement, even if the typefaces didn't match the nixos.org homepage and whatnot.

Agree that the intended stylesheets should be applied, but we should make a note to go fix the whitespace in the table of contents. (Where are those stylesheets? I could open a PR.)

9999years avatar Mar 14 '24 06:03 9999years

The default stylesheet is "incomplete", in the sense that certain elements (such as caution blocks) are unstyled and cause confusion when reading the manual.

The way that things worked before is that the manuals were built with the default stylesheets, and the previous homepage added extra stylesheets on top of that to get to the style we know.

With the move to astro, those extra stylesheets were removed and not ported anywhere, so we get only the default ones. I'm working to bring those extra stylesheets into the default, which should make the manual look the way we expect in the astro homepage, and also will have the benefit of showing documentation writers the final state of the manual during development (we just accepted that the manual we saw during development was different).

DanielSidhion avatar Mar 14 '24 07:03 DanielSidhion

I think this is the better solution anyway since people can have the same manual experience locally as they have on the website. 👍 Thanks for working on this. 🙂

thilobillerbeck avatar Mar 14 '24 09:03 thilobillerbeck

@DanielSidhion is fixing this for the NixOS/Nixpkgs manuals in https://github.com/NixOS/nixpkgs/pull/295847! :heart:

infinisil avatar Mar 14 '24 15:03 infinisil

This is mostly resolved by now, thre might be a need for navigating back to the homepage from a manual someday, but at least the styling is consistent and reapplied

thilobillerbeck avatar Jul 02 '24 19:07 thilobillerbeck