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

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

Open
samueldr opened this issue Mar 2, 2024 · 10 comments
Open
Labels
design About the design refresh work new-techstack-beta This tag marks issues with the new tech stack during preview phase

Comments

@samueldr
Copy link
Member

samueldr commented Mar 2, 2024

This also applies to the pills.

image

@samueldr samueldr added the design About the design refresh work label Mar 2, 2024
@thilobillerbeck thilobillerbeck added the new-techstack-beta This tag marks issues with the new tech stack during preview phase label Mar 2, 2024
@thilobillerbeck
Copy link
Collaborator

thilobillerbeck commented Mar 2, 2024

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

@samueldr
Copy link
Member Author

samueldr commented Mar 3, 2024

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.

@asymmetric
Copy link
Contributor

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

@garbas
Copy link
Member

garbas commented Mar 13, 2024

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

@infinisil
Copy link
Member

infinisil commented Mar 13, 2024

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?

@fricklerhandwerk
Copy link
Contributor

fricklerhandwerk commented Mar 13, 2024

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.

@9999years
Copy link

...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.)

@DanielSidhion
Copy link
Member

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).

@thilobillerbeck
Copy link
Collaborator

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. 🙂

@infinisil
Copy link
Member

@DanielSidhion is fixing this for the NixOS/Nixpkgs manuals in NixOS/nixpkgs#295847! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design About the design refresh work new-techstack-beta This tag marks issues with the new tech stack during preview phase
Projects
None yet
Development

No branches or pull requests

8 participants