From e02d736f2e7d0cbbc2d962b09afaa0d8ff5f1c76 Mon Sep 17 00:00:00 2001 From: Gustavo Henrique Santos Souza de Miranda Date: Wed, 21 Jan 2026 17:21:52 -0300 Subject: [PATCH] chore: mark US-029 as complete and update progress log Co-Authored-By: Claude Opus 4.5 --- prd.json | 2 +- progress.txt | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/prd.json b/prd.json index 5c2d0e4..29d01a7 100644 --- a/prd.json +++ b/prd.json @@ -514,7 +514,7 @@ "Verify in browser using dev-browser skill" ], "priority": 29, - "passes": false, + "passes": true, "notes": "" }, { diff --git a/progress.txt b/progress.txt index 26bef88..dab9d29 100644 --- a/progress.txt +++ b/progress.txt @@ -421,3 +421,15 @@ - The useNodesState/useEdgesState hooks with onNodesChange/onEdgesChange handle all deletion state updates - No explicit onNodesDelete callback is needed - the onNodesChange handler covers deletion events --- + +## 2026-01-21 - US-029 +- What was implemented: Select and delete edges functionality +- Files changed: + - src/app/editor/[projectId]/FlowchartEditor.tsx - added onEdgesDelete callback +- **Learnings for future iterations:** + - React Flow 11 edges are clickable and selectable by default (interactionWidth renders invisible interaction area) + - The `deleteKeyCode` prop works for both nodes and edges - same configuration covers both + - onEdgesDelete is optional if you just need state management (onEdgesChange handles removal events) + - onEdgesDelete is useful for additional logic like logging, dirty state tracking, or undo/redo + - Edge selection shows visual highlight via React Flow's built-in styling +---