Skip to content

docs: humanize wording and remove AI-style em dashes#1421

Merged
mathuo merged 1 commit into
v8-branchfrom
claude/v8-docs-humanize-sec4eb
Jul 8, 2026
Merged

docs: humanize wording and remove AI-style em dashes#1421
mathuo merged 1 commit into
v8-branchfrom
claude/v8-docs-humanize-sec4eb

Conversation

@mathuo

@mathuo mathuo commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Description

Reworks prose across the documentation site so it reads as human-written rather than AI-generated. Rebased onto v8-branch and targets v8-branch.

The most visible tell was the em dash: there were ~400 across the v8 doc pages. Every one is removed, replaced with whatever the sentence actually calls for (comma, colon, semicolon, parentheses, or full stop) rather than a mechanical swap. Also dropped a few stock phrasings ("a comprehensive set of events", "a wide variety of", "out of the box", "This unlocks correct behaviour").

Deliberately left alone: British spelling (behaviour, colour, localise), the en dash in numeric ranges like v5.0–5.2, and pre-existing typos in older hand-written sections, since none of those read as AI.

Also adds a Writing Style section to packages/docs/AGENTS.md so future doc edits avoid the same tells.

Knock-on changes: removing em dashes from a few headings changed their slugs, so ## React users — action required became ## React users: action required and the cross-reference anchor in whats-new-v7.mdx was updated to match. The ## Renamed methods heading keeps its explicit {#renamed-methods} anchor, so its inbound link is unaffected.

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactor / cleanup
  • Build / CI / tooling

Affected packages

  • dockview-core
  • dockview (vanilla JS)
  • dockview-react
  • dockview-vue
  • dockview-angular
  • docs

How to test

Prose-only changes; no runtime behaviour is affected. To review:

  • Read the diff for wording and punctuation.
  • Confirm the internal link from What's new in v7 to Migrating to v7 → React users: action required still resolves (its heading slug changed alongside the em-dash removal).

Checklist

  • yarn lint:fix passes
  • yarn format passes (docs package is deliberately excluded from Prettier)
  • npm run gen has been run and generated files are up to date
  • yarn test passes
  • I have added or updated tests where applicable
  • Breaking changes are documented

🤖 Generated with Claude Code

https://claude.ai/code/session_01698iGKTFkcMr51WC1grrvT

Rework prose across the documentation site so it reads as human-written:
replace em dashes with commas, colons, semicolons, parentheses or full
stops as the sentence calls for, and drop stock phrasings
("comprehensive set of", "a wide variety of", "out of the box",
"unlocks correct behaviour"). Update the renamed migration headings and
the cross-reference anchor to keep links working. Also adds a Writing
Style section to the docs AGENTS.md to deter the same phrasings in future.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01698iGKTFkcMr51WC1grrvT
@mathuo mathuo force-pushed the claude/v8-docs-humanize-sec4eb branch from a7cb264 to 2509342 Compare July 8, 2026 22:10
@mathuo mathuo changed the base branch from master to v8-branch July 8, 2026 22:10
@mathuo mathuo merged commit 2fc3365 into v8-branch Jul 8, 2026
7 checks passed
@sonarqubecloud

sonarqubecloud Bot commented Jul 8, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants