render markdown, strip html tags in indexing

[samthor]

Fixes #917
Fixes #924

I know @devnook tried to solve this in #923. I disagree with @robdodson 's comment, I think we should break-word just in case.

Fixes a minor display issue too; if we don't have content on page (which happens for some of the structural pages), fall back to the meta description.

[netlify]

✔️ Deploy Preview for developer-chrome-com ready!

🔨 Explore the source changes: b7961c1

🔍 Inspect the deploy log: https://app.netlify.com/sites/developer-chrome-com/deploys/611c8faee0716900088c3411

😎 Browse the preview: https://deploy-preview-1148--developer-chrome-com.netlify.app

[robdodson]

I know @devnook tried to solve this in #923. I disagree with @robdodson 's comment, I think we should break-word just in case.

What case are you worried about? One that I could think of is if someone tries to have a reallyLongMethodName but I think our site's code style may already cause that to break (I think ... would be good to test).

[robdodson]

Visual comparison with and without break-all.

With break-all every word near the right-hand edge will get broken in a weird way.

If you remove the style then the words wrap naturally

[samthor]

You're demoing break-all? The break-word option, despite its confusing name, avoids breaking words where possible. It only breaks words too long to render at once. The example on MDN is good for this: with break-word, the first two words remain on their own lines, it's only the third really huge word that's split up.

[robdodson]

ah sorry, I didn't notice that you said break-word. I thought you were saying you disagreed with my comment not to use break-all in Ewa's PR. Yeah, break-word is fine.

[devnook]

Is this already rendered content, e.g. with shortcodes rendered in?

[samthor]

yes - templateContent is itself a getter which renders the final page content but only when 11ty is rendering pages. If we get it here, we crash.

[robdodson]

@samthor any idea why the linter is failing? it seems to be missing a module

[samthor]

I expanded the typechecking a tiny bit. I've fixed the other types issues in an extra commit here.