The Phase 3 preview-pane inline overlay and Phase 4 raw-pane gutter
accent together cover the review affordance DiffView was the interim
home for, so the toolbar toggle collapses and the surface goes.
- frontend/src/components/DiffView.jsx deleted.
- RFCView.jsx: drop the DiffView + marked imports, the reviewMode /
reviewHTML state, the toggleReviewMode callback, the "Review changes"
toolbar button, and the reviewMode ternary in render. editorEditable
simplifies to mode === 'contribute' && canContribute. File-header
docstring updated: "DiffView" → "preview-pane tracked-change overlay
(§8.10)".
- App.css: remove the .diff-view-wrapper and .diff-view-empty rules;
relabel the DiffView section header as ChangeTooltip (which lives on
as the Phase 3 overlay's hover affordance). Drop the dead .editor-
content .tiptap rules — Tiptap had no remaining consumers after
DiffView's deletion. The Contribute toolbar still renders the
accepted/pending hint + Phase 6 Chat toggle; only the Review-changes
button is removed.
- MarkdownPreview.jsx: comment updated to drop the DiffView mention
from the scoped-Marked-instance rationale.
- SPEC.md: §8.10 retitled "Tracked-change markup" (was "...and the
review-mode toggle"); the legacy-DiffView paragraph rewritten as a
one-sentence retirement note. §8.1, §8.15, §9.4, §9.5 references to
DiffView / review-mode cleaned up. §19.2's persistent-accepted-change
candidate updated to point at the preview-pane overlay rather than
DiffView.
No DiffView orphans remain (grep confirms .btn-review-toggle is still
the Discuss-mode "Show tracked changes" toggle, ChangeTooltip and
trackedOverlay carry historical comments only, PRView's §19.2 comment
is unrelated). Frontend reload shows no console errors.
Backend integration suite: 125 passed.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Ben Stull
46049d296b