chore: mark US-029 as complete and update progress log
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
parent
4c7a289714
commit
e02d736f2e
2
prd.json
2
prd.json
|
|
@ -514,7 +514,7 @@
|
||||||
"Verify in browser using dev-browser skill"
|
"Verify in browser using dev-browser skill"
|
||||||
],
|
],
|
||||||
"priority": 29,
|
"priority": 29,
|
||||||
"passes": false,
|
"passes": true,
|
||||||
"notes": ""
|
"notes": ""
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
|
|
|
||||||
12
progress.txt
12
progress.txt
|
|
@ -421,3 +421,15 @@
|
||||||
- The useNodesState/useEdgesState hooks with onNodesChange/onEdgesChange handle all deletion state updates
|
- The useNodesState/useEdgesState hooks with onNodesChange/onEdgesChange handle all deletion state updates
|
||||||
- No explicit onNodesDelete callback is needed - the onNodesChange handler covers deletion events
|
- 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
|
||||||
|
---
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue