From ccd88b39d1fc314ac51ae77feadef4aa0a21ff80 Mon Sep 17 00:00:00 2001 From: Gustavo Henrique Santos Souza de Miranda Date: Wed, 21 Jan 2026 17:19:05 -0300 Subject: [PATCH] chore: mark US-028 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 07200b2..5c2d0e4 100644 --- a/prd.json +++ b/prd.json @@ -499,7 +499,7 @@ "Verify in browser using dev-browser skill" ], "priority": 28, - "passes": false, + "passes": true, "notes": "" }, { diff --git a/progress.txt b/progress.txt index cb616df..26bef88 100644 --- a/progress.txt +++ b/progress.txt @@ -409,3 +409,15 @@ - Apply consistent edge styling in both onConnect (new edges) and toReactFlowEdges (loaded edges) - Generate unique edge IDs with nanoid in onConnect callback --- + +## 2026-01-21 - US-028 +- What was implemented: Select and delete nodes functionality +- Files changed: + - src/app/editor/[projectId]/FlowchartEditor.tsx - added deleteKeyCode prop to enable Delete/Backspace key deletion +- **Learnings for future iterations:** + - React Flow has built-in node selection via clicking - no extra configuration needed + - Use `deleteKeyCode={['Delete', 'Backspace']}` prop to enable keyboard deletion + - React Flow automatically removes connected edges when a node is deleted (no manual cleanup needed) + - The useNodesState/useEdgesState hooks with onNodesChange/onEdgesChange handle all deletion state updates + - No explicit onNodesDelete callback is needed - the onNodesChange handler covers deletion events +---