tahoe-lafs/web-landing-page
1
0
Fork 0
Code Issues 4 Pull requests 2 Actions Activity

Remove custom font-family override #23

Merged
hacklschorsch merged 0 commits from no-font-override into main 2025-05-07 07:27:57 +00:00
Conversation 3 Commits - Files changed -
hacklschorsch commented 2025-04-25 10:38:58 +00:00 (Migrated from code.lafs.eval.latfa.net)
Copy link

Minimal-mistakes already uses sane and generally available fonts.

See https://mmistakes.github.io/minimal-mistakes/docs/stylesheets/#font-stacks on how to override font family choice with this theme if we still want to do it and https://infinnie.github.io/blog/2017/systemui.html why not to use 'system-ui' for font-family on web pages.

Minimal-mistakes already uses sane and generally available fonts. See https://mmistakes.github.io/minimal-mistakes/docs/stylesheets/#font-stacks on how to override font family choice with this theme if we still want to do it and https://infinnie.github.io/blog/2017/systemui.html why not to use 'system-ui' for font-family on web pages.
gitea-actions commented 2025-04-25 10:39:32 +00:00 (Migrated from code.lafs.eval.latfa.net)
Copy link

🔨 Docker build success

🔧 Jekyll build success

🚀 Jekyll deploy success

🔗 Preview site here

Pusher: @hacklschorsch, Action: pull_request, Workflow: Jekyll, Run: 170

#### :hammer: Docker build `success` #### :wrench: Jekyll build `success` #### :rocket: Jekyll deploy `success` #### :link: Preview site [here](https://preview.lafs.eval.latfa.net&#x2F;23&#x2F;) *Pusher: @hacklschorsch, Action: `pull_request`, Workflow: `Jekyll`, Run: 170* <!-- pr-commenter-metadata: image-success,build-success,deploy-success,pusher-hacklschorsch-pull_request-170 -->
btlogy (Migrated from code.lafs.eval.latfa.net) reviewed 2025-04-25 11:50:03 +00:00
btlogy commented 2025-05-02 09:00:17 +00:00 (Migrated from code.lafs.eval.latfa.net)
Copy link

@hacklschorsch: I do not mind dropping this font, but it was the one used by the mockup (see #12).

@hacklschorsch: I do not mind dropping this font, but it was the one used by the mockup (see #12).
btlogy commented 2025-05-07 07:27:53 +00:00 (Migrated from code.lafs.eval.latfa.net)
Copy link

As commented here, system-ui was a mistake indeed.

As commented [here](https://code.lafs.eval.latfa.net/tahoe-lafs/web-landing-page/issues/12#issuecomment-110182), `system-ui` was a mistake indeed.
Sign in to join this conversation.
Reviewers
No reviewers
btlogy
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: tahoe-lafs/web-landing-page#23
No description provided.
Delete branch "no-font-override"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?