Page MenuHomePhabricator

[landing] Define `--font-stack` for `landing`
AbandonedPublic

Authored by abosh on Jun 24 2022, 12:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 30, 5:36 AM
Unknown Object (File)
Sat, Jun 29, 8:05 PM
Unknown Object (File)
Tue, Jun 25, 11:37 PM
Unknown Object (File)
Fri, Jun 21, 7:11 PM
Unknown Object (File)
Fri, Jun 21, 7:10 PM
Unknown Object (File)
Fri, Jun 21, 6:52 PM
Unknown Object (File)
Tue, Jun 18, 9:21 AM
Unknown Object (File)
May 10 2024, 1:26 PM
Subscribers

Details

Reviewers
atul
tomek
Summary

Relevant Linear issue with full context here. Defined new --font-stack global CSS variable using the :root pseudo-class and replaced all instances of the same font usages throughout landing ('IBM Plex Sans', sans-serif).

Read more about CSS global variables.

Test Plan

Made sure everything on landing looked the same before and after the variable was introduced. So, this included the header, footer, subscription form, read docs button, and info blocks.

Diff Detail

Repository
rCOMM Comm
Branch
fontstack (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Should we rename --font-stack to --sans-serif? We can then create global --monospace, --serif, etc as well.

This revision is now accepted and ready to land.Jun 24 2022, 12:51 PM

I agree. Having some sort of git issue, so abandoning this revision, but the new revision with the renamed font-stack is here: D4356.