feat: add wire undo/redo with snapshot-based history stack
- Snapshot-based undo/redo (full wires array per step, max 50 entries) - Keyboard shortcuts: Ctrl+Z (undo), Ctrl+Shift+Z (redo) - Toolbar buttons in canvas header (visible on mobile via separate undo-controls CSS class that isn't hidden by mobile media query) - All 4 wire mutations (add, remove, update, finish) push snapshots - selectedWireId reset on undo/redo to avoid stale referencesmaster
parent
cd85b78b75
commit
52321dffe8
|
|
@ -183,7 +183,8 @@
|
||||||
}
|
}
|
||||||
|
|
||||||
/* ── Zoom controls ───────────────────────────────── */
|
/* ── Zoom controls ───────────────────────────────── */
|
||||||
.zoom-controls {
|
.zoom-controls,
|
||||||
|
.undo-controls {
|
||||||
display: flex;
|
display: flex;
|
||||||
align-items: center;
|
align-items: center;
|
||||||
gap: 2px;
|
gap: 2px;
|
||||||
|
|
@ -208,11 +209,16 @@
|
||||||
flex-shrink: 0;
|
flex-shrink: 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
.zoom-btn:hover {
|
.zoom-btn:hover:not(:disabled) {
|
||||||
background: #3a3a3a;
|
background: #3a3a3a;
|
||||||
color: #ccc;
|
color: #ccc;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
.zoom-btn:disabled {
|
||||||
|
opacity: 0.3;
|
||||||
|
cursor: default;
|
||||||
|
}
|
||||||
|
|
||||||
.zoom-level {
|
.zoom-level {
|
||||||
min-width: 42px;
|
min-width: 42px;
|
||||||
text-align: center;
|
text-align: center;
|
||||||
|
|
@ -279,6 +285,51 @@
|
||||||
background: rgba(255, 255, 255, 0.35);
|
background: rgba(255, 255, 255, 0.35);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* ── Wire selected banner (delete/deselect) ─────── */
|
||||||
|
.wire-selected-banner {
|
||||||
|
position: absolute;
|
||||||
|
bottom: 12px;
|
||||||
|
left: 50%;
|
||||||
|
transform: translateX(-50%);
|
||||||
|
z-index: 100;
|
||||||
|
background: rgba(60, 60, 70, 0.92);
|
||||||
|
color: #fff;
|
||||||
|
padding: 8px 16px;
|
||||||
|
border-radius: 8px;
|
||||||
|
display: flex;
|
||||||
|
align-items: center;
|
||||||
|
gap: 12px;
|
||||||
|
font-size: 13px;
|
||||||
|
box-shadow: 0 2px 10px rgba(0, 0, 0, 0.5);
|
||||||
|
backdrop-filter: blur(4px);
|
||||||
|
white-space: nowrap;
|
||||||
|
pointer-events: auto;
|
||||||
|
}
|
||||||
|
|
||||||
|
.wire-selected-banner button {
|
||||||
|
color: #fff;
|
||||||
|
border: 1px solid rgba(255, 255, 255, 0.4);
|
||||||
|
border-radius: 4px;
|
||||||
|
padding: 4px 14px;
|
||||||
|
cursor: pointer;
|
||||||
|
font-size: 12px;
|
||||||
|
font-weight: 600;
|
||||||
|
background: rgba(255, 255, 255, 0.2);
|
||||||
|
transition: background 0.15s;
|
||||||
|
}
|
||||||
|
|
||||||
|
.wire-selected-banner button:hover {
|
||||||
|
background: rgba(255, 255, 255, 0.35);
|
||||||
|
}
|
||||||
|
|
||||||
|
.wire-selected-banner button.btn-danger {
|
||||||
|
background: rgba(220, 53, 69, 0.7);
|
||||||
|
}
|
||||||
|
|
||||||
|
.wire-selected-banner button.btn-danger:hover {
|
||||||
|
background: rgba(220, 53, 69, 0.9);
|
||||||
|
}
|
||||||
|
|
||||||
/* ── Mobile-friendly adjustments ────────────────── */
|
/* ── Mobile-friendly adjustments ────────────────── */
|
||||||
@media (max-width: 768px) {
|
@media (max-width: 768px) {
|
||||||
.canvas-header {
|
.canvas-header {
|
||||||
|
|
@ -325,7 +376,8 @@
|
||||||
display: none;
|
display: none;
|
||||||
}
|
}
|
||||||
|
|
||||||
.wire-mode-banner {
|
.wire-mode-banner,
|
||||||
|
.wire-selected-banner {
|
||||||
font-size: 12px;
|
font-size: 12px;
|
||||||
padding: 6px 12px;
|
padding: 6px 12px;
|
||||||
gap: 8px;
|
gap: 8px;
|
||||||
|
|
|
||||||
|
|
@ -235,6 +235,8 @@ interface SimulatorState {
|
||||||
wires: Wire[];
|
wires: Wire[];
|
||||||
selectedWireId: string | null;
|
selectedWireId: string | null;
|
||||||
wireInProgress: WireInProgress | null;
|
wireInProgress: WireInProgress | null;
|
||||||
|
wireUndoStack: Wire[][];
|
||||||
|
wireRedoStack: Wire[][];
|
||||||
addWire: (wire: Wire) => void;
|
addWire: (wire: Wire) => void;
|
||||||
removeWire: (wireId: string) => void;
|
removeWire: (wireId: string) => void;
|
||||||
updateWire: (wireId: string, updates: Partial<Wire>) => void;
|
updateWire: (wireId: string, updates: Partial<Wire>) => void;
|
||||||
|
|
@ -248,6 +250,8 @@ interface SimulatorState {
|
||||||
cancelWireCreation: () => void;
|
cancelWireCreation: () => void;
|
||||||
updateWirePositions: (componentId: string) => void;
|
updateWirePositions: (componentId: string) => void;
|
||||||
recalculateAllWirePositions: () => void;
|
recalculateAllWirePositions: () => void;
|
||||||
|
undoWire: () => void;
|
||||||
|
redoWire: () => void;
|
||||||
|
|
||||||
// ── Serial monitor ──────────────────────────────────────────────────────
|
// ── Serial monitor ──────────────────────────────────────────────────────
|
||||||
toggleSerialMonitor: () => void;
|
toggleSerialMonitor: () => void;
|
||||||
|
|
@ -1085,6 +1089,8 @@ export const useSimulatorStore = create<SimulatorState>((set, get) => {
|
||||||
],
|
],
|
||||||
selectedWireId: null,
|
selectedWireId: null,
|
||||||
wireInProgress: null,
|
wireInProgress: null,
|
||||||
|
wireUndoStack: [],
|
||||||
|
wireRedoStack: [],
|
||||||
|
|
||||||
addComponent: (component) => set((state) => ({ components: [...state.components, component] })),
|
addComponent: (component) => set((state) => ({ components: [...state.components, component] })),
|
||||||
|
|
||||||
|
|
@ -1114,14 +1120,22 @@ export const useSimulatorStore = create<SimulatorState>((set, get) => {
|
||||||
|
|
||||||
setComponents: (components) => set({ components }),
|
setComponents: (components) => set({ components }),
|
||||||
|
|
||||||
addWire: (wire) => set((state) => ({ wires: [...state.wires, wire] })),
|
addWire: (wire) => set((state) => ({
|
||||||
|
wireUndoStack: [...state.wireUndoStack, state.wires].slice(-50),
|
||||||
|
wireRedoStack: [],
|
||||||
|
wires: [...state.wires, wire],
|
||||||
|
})),
|
||||||
|
|
||||||
removeWire: (wireId) => set((state) => ({
|
removeWire: (wireId) => set((state) => ({
|
||||||
|
wireUndoStack: [...state.wireUndoStack, state.wires].slice(-50),
|
||||||
|
wireRedoStack: [],
|
||||||
wires: state.wires.filter((w) => w.id !== wireId),
|
wires: state.wires.filter((w) => w.id !== wireId),
|
||||||
selectedWireId: state.selectedWireId === wireId ? null : state.selectedWireId,
|
selectedWireId: state.selectedWireId === wireId ? null : state.selectedWireId,
|
||||||
})),
|
})),
|
||||||
|
|
||||||
updateWire: (wireId, updates) => set((state) => ({
|
updateWire: (wireId, updates) => set((state) => ({
|
||||||
|
wireUndoStack: [...state.wireUndoStack, state.wires].slice(-50),
|
||||||
|
wireRedoStack: [],
|
||||||
wires: state.wires.map((w) => w.id === wireId ? { ...w, ...updates } : w),
|
wires: state.wires.map((w) => w.id === wireId ? { ...w, ...updates } : w),
|
||||||
})),
|
})),
|
||||||
|
|
||||||
|
|
@ -1173,11 +1187,38 @@ export const useSimulatorStore = create<SimulatorState>((set, get) => {
|
||||||
waypoints,
|
waypoints,
|
||||||
color,
|
color,
|
||||||
};
|
};
|
||||||
set((state) => ({ wires: [...state.wires, newWire], wireInProgress: null }));
|
set((state) => ({
|
||||||
|
wireUndoStack: [...state.wireUndoStack, state.wires].slice(-50),
|
||||||
|
wireRedoStack: [],
|
||||||
|
wires: [...state.wires, newWire],
|
||||||
|
wireInProgress: null,
|
||||||
|
}));
|
||||||
},
|
},
|
||||||
|
|
||||||
cancelWireCreation: () => set({ wireInProgress: null }),
|
cancelWireCreation: () => set({ wireInProgress: null }),
|
||||||
|
|
||||||
|
undoWire: () => set((state) => {
|
||||||
|
if (state.wireUndoStack.length === 0) return state;
|
||||||
|
const prev = state.wireUndoStack[state.wireUndoStack.length - 1];
|
||||||
|
return {
|
||||||
|
wireUndoStack: state.wireUndoStack.slice(0, -1),
|
||||||
|
wireRedoStack: [...state.wireRedoStack, state.wires].slice(-50),
|
||||||
|
wires: prev,
|
||||||
|
selectedWireId: null,
|
||||||
|
};
|
||||||
|
}),
|
||||||
|
|
||||||
|
redoWire: () => set((state) => {
|
||||||
|
if (state.wireRedoStack.length === 0) return state;
|
||||||
|
const next = state.wireRedoStack[state.wireRedoStack.length - 1];
|
||||||
|
return {
|
||||||
|
wireRedoStack: state.wireRedoStack.slice(0, -1),
|
||||||
|
wireUndoStack: [...state.wireUndoStack, state.wires].slice(-50),
|
||||||
|
wires: next,
|
||||||
|
selectedWireId: null,
|
||||||
|
};
|
||||||
|
}),
|
||||||
|
|
||||||
updateWirePositions: (componentId) => {
|
updateWirePositions: (componentId) => {
|
||||||
set((state) => {
|
set((state) => {
|
||||||
const component = state.components.find((c) => c.id === componentId);
|
const component = state.components.find((c) => c.id === componentId);
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue