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 +---