[myieye]

[Claude, autonomous]

Stacked on #2882. The activity change preview now shows what a change did, not the whole entity: only the changed fields, under an anchor line that says which entry, sense or example it is. Unchanged fields sit behind a per-change "Show N unchanged fields" toggle.

What each change type shows (activity preview, mode="changes")

Change	Shows
Entry edit	Anchor: best headword as an inline before → after diff, plus the homograph number as a subscript (a 0 → 1 change shows the added ₁ highlighted). Then the changed fields only (morph type still appears only when it changed).
Sense edit	Anchor: best gloss (view's analysis WSs first), or the definition when neither side has a gloss. Then the changed fields.
Example edit	Anchor: best sentence (view's vernacular WSs first). Then the changed fields.
Create / delete	Every non-empty field. Nothing counts as unchanged, so there's no toggle.
Nothing visible changed (formatting-only rich text, id/deletedAt-style fields)	Anchor, a muted "No visible changes" line styled like "Preview not available", and the full entity.
Objects (part of speech, writing system, custom view, ...)	Unchanged (see deviations).

• "Show N unchanged fields" / "Hide unchanged fields" is a button with aria-expanded (DiffCollapsed.svelte). It uses $plural the way the repo does, and it's hidden when N is 0. Its state lives in each primitive instance, so each change in a commit collapses on its own. It isn't persisted, and a change scrolled out of the virtual list resets.
• The primitives take mode: 'changes' | 'full' and default to 'full'. Existing callers render the same fields as before, plus the anchor, which always shows, as the spec asked.
• changedFieldIds(fields, before, after, compareAs) sits next to showChangedFields in changed-fields.ts and reuses valueChanged. showChangedFields is now built on it. A small changesOnly helper narrows the fields and counts the hidden ones.

Desktop	Expanded	Phone (390px)

Deviations from the spec, and why

• N counts only unchanged fields that have a value. Unchanged empty fields aren't counted, because revealing them shows nothing. That's also what makes creates and deletes come out with N = 0, as the spec expected.
• "No visible changes" only applies to edits. A create or delete is a visible change even when every field is empty, so an empty create shows its full (empty) form without the note. In 'full' mode the note never shows, so the existing behaviour stays the same.
• No sense number in the sense anchor. The sense snapshot doesn't carry its position. The only source is the current entry's sense order, which can disagree with the order when the change was made. That isn't cheap and could mislead.
• DiffFieldRows (objects) has no 'changes' mode. Filtering the rows would be trivial. But without an anchor, a writing-system font change would show only "Font", with nothing saying which writing system it was. Adding anchors per object type goes beyond the trivial case, so I left it for a follow-up.
• The anchor uses viewBestHeadword, the same headword lists show. When a citation form exists it wins, so a lexeme-form edit can leave the anchor unchanged (visible in the first story).
• An empty grid is hidden. When no field is left to show (e.g. a homograph-only change), the empty sub-grid is hidden so its gap doesn't leave a blank band.

Homograph numbers in DiffAnchor.svelte now use the shared HomographNumber component from #2881 (now in this stack), wrapped in the diff colouring.

Test plan

• changed-fields.test.ts: new cases for changedFieldIds (shown and hidden fields, formatting-only edits, compareAs, create/delete) and changesOnly (hiding fields and counting only those with a value).
• diff-field-coverage.test.ts: every IEntry / ISense / IExampleSentence key is either diffed (a view field, or morphType / homographNumber shown outside the view) or in an explicit not-visible list. A new backend field fails until someone decides.
• svelte-check, eslint, vitest unit project (261 passed), i18n extract stable (translator comments added for the 3 new msgids).
• Playwright entry-edit-persists and entries-list, chromium: 17 passed. The demo has no activity data, so no UI test covers the preview itself.
• Storybook editor/diff/entities: the new "Changes only" story and the existing full story, checked with Playwright at 1100px and 390px. No horizontal scroll; toggling one change leaves the others collapsed; aria-expanded flips.
• Manual: open Activity in FW Lite on a project with edits, creates, deletes and a formatting-only edit.

---

Generated by Claude Code

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[myieye]

[Claude, autonomous]

Combined into #2889 with #2885 and #2886.

---

Generated by Claude Code