Design: a review verdict's body uses the design system, and review-body leaves §5d #3628

A review verdict's body now sits in a .card holding a .prose.comment-body, and .review-body is gone. I did not run the type check: bun x tsc --noEmit failed here with "bun is unable to write files to tempdir". I also did not take the before/after screenshots (light and dark), so that Done item is unmet.

What changed

• src/web/studio/change.ts renders the verdict body in that card.
• One rule in src/web/style/components.ts keeps the card full-width in the timeline row (flex-basis: 100%) and sets its text colour and size. Another sets the body padding. Both are scoped to .tl-item[data-kind="event"], so they add no new class.
• .review-body and its two paragraph rules are removed from src/web/style/content.ts, and the "review verdicts" bullet is gone from docs/DESIGN.md §5d.
• The only other review-body hits are id="review-body" and for="review-body" on the review form's textarea in src/web/pages/pull.ts. They are an element id, not the class, so I left them.
• test/review-verdict-body.test.ts opens a pull with a "commented" review (fine and a list) and checks four things:

- the markdown still renders; - the body uses prose comment-body; - no review-body class appears in the HTML; - it fails without the change.

Checks

• The new test passes, and so do the design tests and bun scripts/design-lint.ts. I did not run the whole bun test suite.

Left out

• The test uses a "commented" review rather than approved and changes-requested ones. Those states share the same code path in change.ts, and an author can't approve their own pull.

To try it, open a pull that has a review with a body and check the body sits in a bordered card under the verdict line.

Closes #3619

A review verdict's body now sits in a .card holding a .prose.comment-body, and .review-body is gone. I did not run the type check: bun x tsc --noEmit failed here with "bun is unable to write files to tempdir". I also did not take the before/after screenshots (light and dark), so that Done item is unmet.

What changed

• src/web/studio/change.ts renders the verdict body in that card.
• One rule in src/web/style/components.ts keeps the card full-width in the timeline row (flex-basis: 100%) and sets its text colour and size. Another sets the body padding. Both are scoped to .tl-item[data-kind="event"], so they add no new class.
• .review-body and its two paragraph rules are removed from src/web/style/content.ts, and the "review verdicts" bullet is gone from docs/DESIGN.md §5d.
• The only other review-body hits are id="review-body" and for="review-body" on the review form's textarea in src/web/pages/pull.ts. They are an element id, not the class, so I left them.
• test/review-verdict-body.test.ts opens a pull with a "commented" review (fine and a list) and checks four things:

- the markdown still renders; - the body uses prose comment-body; - no review-body class appears in the HTML; - it fails without the change.

Checks

• The new test passes, and so do the design tests and bun scripts/design-lint.ts. I did not run the whole bun test suite.

Left out

• The test uses a "commented" review rather than approved and changes-requested ones. Those states share the same code path in change.ts, and an author can't approve their own pull.

To try it, open a pull that has a review with a body and check the body sits in a bordered card under the verdict line.