Skip to content

Atkinson font added here for now#824

Open
timea-solid wants to merge 2 commits into
stagingfrom
font
Open

Atkinson font added here for now#824
timea-solid wants to merge 2 commits into
stagingfrom
font

Conversation

@timea-solid

Copy link
Copy Markdown
Member

I added the font for now here.
However, it seems like it is overwritten by the --wa-font-family-body: ui-sans-serif, system-ui, sans-serif;

I also need to delete the font in mashlib light.css.

@timea-solid timea-solid requested a review from NoelDeMartin July 1, 2026 11:29
@timea-solid timea-solid self-assigned this Jul 1, 2026
@timea-solid timea-solid marked this pull request as draft July 1, 2026 11:29

@NoelDeMartin NoelDeMartin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can work for now, and maybe we could even release it like this... But we should discuss whether we want to add a dependency with google for the fonts. People using Solid will be particularly privacy-sensitive, so I don't know if they'll like seeing that every time they open SolidOS, they'll be pinging google's servers.

As per the web awesome fonts variables, I think it should be fine to simply define --wa-font-family-body as well. See their theming docs.

@timea-solid timea-solid moved this to In review in SolidOS NLNet UI Jul 2, 2026
@timea-solid timea-solid marked this pull request as ready for review July 2, 2026 11:25
@timea-solid timea-solid requested a review from NoelDeMartin July 2, 2026 11:28
@timea-solid

Copy link
Copy Markdown
Member Author

Consider also merging SolidOS/mashlib#441 (non blocking)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

2 participants