This article doesn’t fill me with confidence.
—⁂—
> Styling Markdown
As others have said, this is nothing to do with Markdown. It’s regular HTML styling.
—⁂—
> Paragraph <p>
… proceeds to talk about CSS Custom Properties (misnaming them CSS Variables—I wouldn’t mind “CSS variables” so much, but the capital V implies a proper noun), which are utterly irrelevant to styling, and body styles, saying that <p> then needs nothing extra.
body {
font-family: ui-sans-serif, "Source Sans Pro", "Helvetica", system-ui;
line-height: 1.75;
}
No, no,
no.
1. ui-sans-serif and system-ui are inappropriate here. UI fonts are not designed for long-form content, and in some OS/language combinations you’ll get an absolutely obnoxious result. Just use `sans-serif`. Or `"Source Sans Pro", sans-serif` if you’re loading that as a web font.
2. 1.75 is unreasonably spacious, especially on small displays. If you’re choosing a single value, you probably shouldn’t go beyond 1.5.
—⁂—
h1, h2, h3, h4, h5, h6 {
overflow-wrap: anywhere;
}
If you think this is worthwhile, you should probably apply it to more than just headings. You may argue that headings’ larger sizes make overflow within a single word more likely, but in my experience it happens much more often in body text, when you do things like write a URL out literally.
—⁂—
img, picture {
object-fit: scale-down;
}
Unnecessary, given `height: auto`. Aside: I believe this is the first time I’ve ever seen scale-down used.
> If you are using a framework like Astro, to center the image you will want to add margin-inline: auto and display: block to center the image.
What has this to do with Astro!? Just as Markdown is a red herring or worse, Astro is irrelevant here.
—⁂—
ol, ul {
margin-left: 1em;
}
I’m
guessing you haven’t used a CSS reset, because otherwise you’d have been adding block margins to things like headings and paragraphs. So, you’ve still got the user-agent stylesheet’s `padding-inline-start: 40px`. And padding is definitely more semantically reasonable than margin, when you consider how the list item markers are placed outside—if you were putting border on the list itself, you’d want the markers to sit inside that border.
So, at present you’ve got 1em + 40px (≈56px) of indent on list item bodies. This is way too much. Even the 40px is too much. But if you forgot to mention that you were zeroing the padding-inline-start: the 1em by itself (≈16px) is too little for numbered lists, double-digit markers will typically overflow (and in some other language or font configurations, even the 1em will be overflowing). I’d suggest 1.5em (≈24px) as the smallest reasonable value for English text with decimal markers.
li {
overflow-wrap: anywhere;
}
As before: ditch this, and if you want its effect, apply it more generally.
—⁂—
> "There's never a space under paintings in a gallery where someone writes their opinion,"
Argh! The image of this has the " straight quotes turned into the
wrong curly quotes! ”…“.
> <footer>— Natalie Dee</footer>
Possibly worth mentioning it’s
relatively common to have `blockquote footer::before { content: "— "; }`.
—⁂—
td {
border: 1px solid green;
padding: 0.5rem 1rem;
}
th {
border: 1px solid green;
padding: 0.75rem 1rem;
font-weight: 600;
}
Could have done `td, th` for the first rule and deduplicated.
background-color: grey; //different background color applied to every second row
No such thing as // comments in CSS. Write /* … */. (There’s another case like it later.)
Also it’s worthwhile mentioning the name for this technique: zebra stripes.
—⁂—
background-color: blue !important;
There’s almost never a good reason to use !important. Definitely I can’t see one here. If you’re using it to override inline styles from your “Astro Expressive Code” or similar: don’t, fix it in its configuration instead.
color: var(--color-side-accent);
font-size: var(--variable-font-size);
It grates that, where normally you’re using obviously placeholder colours, just
occasionally you use variables, sometimes gratuitous unnecessary ones.
code,
code span,
pre {
This is not a well-thought out rule set. Consider each declaration inside and whether it should be applied to all three of these selectors. Most of them shouldn’t (though they’re mostly harmless in practice).
One final remark in this code styling stuff: I find it unreasonably common for this sort of page to apply conflicting sets of styles, with the base styles applied to the HTML actually illegible (e.g. white on white), relying on classes like <html class="light"> being added by JavaScript to make things legible.