Review 5329

#5329 — docs: replace em dashes in MediaTheme and cli-integrations prose

josephfarina · MERGED · view on GitHub

Verdict: approve

Head reviewed: dad76d4295efa38a5319b14d6f421903c6030f0b

Problem

MediaTheme's mode/fallback descriptions and the cli-integrations reference doc used em dashes for asides and clause breaks — the house's AI-slop tell.

Solution

Each aside recast with the punctuation that actually fits: parentheses for parentheticals, a colon or semicolon for clause breaks. Prose only.

Impact

Docsite readers and anyone reading prop descriptions in the CLI or the component docs. No runtime code touched.

API

No API change.

Theme targets

No new theme targets.

Breaking

• API — no. Visual — no, prose only. Theme — no.

Performance & resources

Nothing — no runtime code in the diff.

Risk class

Low-risk — no new API surface, no behavior regression, no performance regression.

Judgement

approve — checked each recast against the original; no technical claim moved. The 3:1 contrast figure, the useImageMode sampling note and the replaces/extends semantics all survive the rewrite intact.

The review, as posted

Thanks — checked each recast against the original, no technical claim moved.

What changed before posting

Posted as drafted.