mirror of
https://github.com/OFFIS-ESC/constellation-analyzer
synced 2026-01-26 23:43:40 +00:00
Fixes incorrect undo behavior when creating groups with actors. Previously, history was captured AFTER mutations, causing undo to restore a state that included the group. This fix moves history capture to BEFORE mutations, making it consistent with all other operations (addNode, deleteNode, addEdge, etc.). Changes: - Move pushToHistory() call before mutations in createGroupWithActors - Add detailed comment explaining the timing - Update refactoring plan with implementation status Impact: - Undo now correctly removes groups and ungroups actors - Behavior consistent with all other operations - No breaking changes to existing functionality Testing: - TypeScript compilation verified (npx tsc --noEmit) - Manual testing plan documented in PHASE_4_1_TEST_PLAN.md - 6 test cases defined for comprehensive verification Related: Phase 4.1 of STATE_MANAGEMENT_REFACTORING_PLAN.md 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| DIRECTIONAL_RELATIONSHIPS_UX_SPEC.md | ||
| IMPLEMENTATION_CHECKLIST.md | ||
| KEYBOARD_SHORTCUTS.md | ||
| MULTI_FILE_PLAN.md | ||
| PERSISTENCE_PLAN.md | ||
| PHASE_4_1_TEST_PLAN.md | ||
| PROJECT_SUMMARY.md | ||
| SIDE_PANELS_UX_DESIGN.md | ||
| STATE_MANAGEMENT_REFACTORING_PLAN.md | ||
| TEMPORAL_ANALYSIS_SUMMARY.md | ||
| TEMPORAL_QUICK_START.md | ||
| TEMPORAL_SCENARIO_IMPLEMENTATION_PLAN.md | ||
| TEMPORAL_SCENARIO_UX_CONCEPT.md | ||
| UNDO_REDO_IMPLEMENTATION.md | ||
| UX_ANALYSIS.md | ||
| UX_ANALYSIS_ACTOR_TYPE_SETTINGS.md | ||
| UX_CONCEPT_MULTI_VERSION_GRAPH.md | ||
| VISUAL_EXAMPLES.md | ||
| WORKSPACE_PERSISTENCE.md | ||