From fe8349819ff7380745408f761e8f8281b38ab6a0 Mon Sep 17 00:00:00 2001 From: Justin Paul Date: Thu, 11 Jun 2026 09:33:19 -0400 Subject: [PATCH] docs: note the spouse-layout fix is upstreamed (donatso/family-chart#105) Co-Authored-By: Claude Opus 4.8 (1M context) Signed-off-by: Justin Paul --- CLAUDE.md | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/CLAUDE.md b/CLAUDE.md index 1a8ed9c..917537c 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -88,7 +88,11 @@ The tree view uses **family-chart** (d3-based). Two adjustments live in the repo To change a patch: edit the file(s) under `node_modules/family-chart/dist/`, then `cd frontend && npx patch-package family-chart` to regenerate, and verify with `npx patch-package --error-on-fail`. -**Upstream these.** Both are general library bugfixes, not app-specific. The `cardToMiddle` fix is submitted — **donatso/family-chart#103** (issue **#102**). The spouse-layout fix still needs upstreaming; do it when there's time. When a fixed release ships, drop the corresponding patch hunk **and** remove any in-app compensation (e.g. the `cardToMiddle` caller in `tree/page.tsx` passes raw `y` precisely because the patch fixes it — pre-scaling there too would double-correct). +**Upstreamed.** Both are general library bugfixes, not app-specific, and are submitted upstream: +- `cardToMiddle` vertical centering — **donatso/family-chart#103** (issue **#102**). +- Multi-spouse centered layout — **donatso/family-chart#105** (issue **#104**). + +If either is merged + released, bump `family-chart`, drop the corresponding patch hunk, **and** remove any in-app compensation (e.g. the `cardToMiddle` caller in `tree/page.tsx` passes raw `y` precisely because the patch fixes it — pre-scaling there too would double-correct). Until then, keep the patch. ## License & contribution terms -- 2.52.0