[landing] Remove `font-size` in `global.css:html`
ClosedPublic
Actions

Authored by atul on Mar 15 2022, 8:56 AM.

Details

Reviewers

• benschac
tomek
ashoat

Commits

rCOMMd5bde70ac237: [landing] Remove `font-size` in `global.css:html`

Summary

We don't want to set a changing font-size in the root (html selector) because that affects what 1rem is downstream. My understanding is that the root should have 1rem = 16px unless the user modifies their browser settings. We should hold this constant and modify font-size dynamically for <p>, <h1>, etc. because they won't all necessarily scale together linearly. This should make the math simpler.

Subsequent diffs will fix things for each element.

Test Plan

NA, observed that 1rem = 16px

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Mar 15 2022, 8:56 AM

Herald added subscribers: • karol, • adrian. · View Herald TranscriptMar 15 2022, 8:56 AM

merge background and background-attachment

landing/global.css
17 ↗	(On Diff #10391)	snuck in this change to use shorthand

Harbormaster completed remote builds in B7380: Diff 10390.Mar 15 2022, 9:01 AM

Harbormaster completed remote builds in B7381: Diff 10391.Mar 15 2022, 9:07 AM

atul requested review of this revision.Mar 15 2022, 9:07 AM

• benschac accepted this revision.Mar 15 2022, 9:35 AM

• benschac added inline comments.

landing/global.css
17 ↗	(On Diff #10391)	Can you screen shot this? Does it change the purple linear-gradient?

This revision is now accepted and ready to land.Mar 15 2022, 9:35 AM

• benschac added inline comments.Mar 15 2022, 9:35 AM

landing/global.css
17 ↗	(On Diff #10391)	Actually, think this would be better as its own diff. That's just me.

atul added a child revision: D3432: [landing] Bring back "fluid" `font-size` to `<p>`.Mar 15 2022, 10:43 AM

atul added 1 blocking reviewer(s): ashoat.Mar 15 2022, 1:27 PM

This revision now requires review to proceed.Mar 15 2022, 1:27 PM

atul added a child revision: D3392: [landing] Keep hero `TextLoop` to two lines.Mar 15 2022, 7:05 PM

ashoat accepted this revision.Mar 15 2022, 8:32 PM

This revision is now accepted and ready to land.Mar 15 2022, 8:32 PM

Actually, think this would be better as its own diff. That's just me. (@benschac)

That's fair. I think it's fine to sneak a noop change into a diff now and then if it's in immediate vicinity (in this case the same css selector block) and explicitly flagged for the reviewer, but can be more disciplined about separating things in the future.