Add four columns option to {.people-rows}

[rakyi]

Also fixes some unrelated styling issues.

[owidbot]

Quick links (staging server):

Site Dev	Site Preview	Admin	Wizard	Docs

Login: ssh owid@staging-site-people-four-cols

• Site-screenshots: https://github.com/owid/site-screenshots/compare/people-four-cols
• SVG tester: https://github.com/owid/owid-grapher-svgs/compare/people-four-cols

SVG tester:

Number of differences (default views): 0 ✅
Number of differences (all views): 0 ✅

Edited: 2024-12-10 18:57:28 UTC
Execution time: 1.20 seconds

[ikesau]

LGTM!

I didn't test all the new CSS overrides, presumably we can do that in a final pass of the draft pages.

[ikesau]

Is this a case we actually want to support? It looks pretty squished on mobile when the columns don't collapse - should columns default to 2 instead?

[ikesau]

Not a big deal, but I think you could also write this as

+ .article-block__side-by-side {
  margin-top: a good value;
}

if you wanted to avoid the :has selector 🙂