feat: add pin hover tooltip during wire aiming and implement null-safety checks for wire rendering

This commit is contained in:
a2nr 2026-04-10 13:59:44 +07:00
parent 1973b0adbc
commit a11dd87d5a
4 changed files with 64 additions and 17 deletions

View File

@ -230,6 +230,7 @@ export const SimulatorCanvas = () => {
const currentTouchRef = useRef({ x: 0, y: 0 }); // latest screen position for timeout callback const currentTouchRef = useRef({ x: 0, y: 0 }); // latest screen position for timeout callback
const [wireAiming, setWireAiming] = useState(false); // React state for rendering (crosshair visible) const [wireAiming, setWireAiming] = useState(false); // React state for rendering (crosshair visible)
const [aimPosition, setAimPosition] = useState<{ x: number; y: number } | null>(null); // world coords of crosshair const [aimPosition, setAimPosition] = useState<{ x: number; y: number } | null>(null); // world coords of crosshair
const [aimHoveredPinName, setAimHoveredPinName] = useState<string | null>(null); // Name of pin hovered by crosshair
// Convert viewport coords to world (canvas) coords // Convert viewport coords to world (canvas) coords
const toWorld = useCallback((screenX: number, screenY: number) => { const toWorld = useCallback((screenX: number, screenY: number) => {
@ -430,6 +431,11 @@ export const SimulatorCanvas = () => {
const aimX = world.x; const aimX = world.x;
const aimY = world.y + AIMING_OFFSET_Y; const aimY = world.y + AIMING_OFFSET_Y;
setAimPosition({ x: aimX, y: aimY }); setAimPosition({ x: aimX, y: aimY });
const snapDist = AIMING_PIN_SNAP_DIST / zoomRef.current;
const nearPin = findNearestPin(aimX, aimY, snapDist);
setAimHoveredPinName(nearPin ? nearPin.pinName : null);
useSimulatorStore.getState().updateWireInProgress(aimX, aimY); useSimulatorStore.getState().updateWireInProgress(aimX, aimY);
}, AIMING_LONG_PRESS_MS); }, AIMING_LONG_PRESS_MS);
} }
@ -495,7 +501,13 @@ export const SimulatorCanvas = () => {
setWireAiming(true); setWireAiming(true);
if (navigator.vibrate) navigator.vibrate(30); if (navigator.vibrate) navigator.vibrate(30);
const world = toWorld(currentTouchRef.current.x, currentTouchRef.current.y); const world = toWorld(currentTouchRef.current.x, currentTouchRef.current.y);
setAimPosition({ x: world.x, y: world.y + AIMING_OFFSET_Y }); const aimX = world.x;
const aimY = world.y + AIMING_OFFSET_Y;
setAimPosition({ x: aimX, y: aimY });
const snapDist = AIMING_PIN_SNAP_DIST / zoomRef.current;
const nearPin = findNearestPin(aimX, aimY, snapDist);
setAimHoveredPinName(nearPin ? nearPin.pinName : null);
}, AIMING_LONG_PRESS_MS); }, AIMING_LONG_PRESS_MS);
} }
isPanningRef.current = true; isPanningRef.current = true;
@ -572,6 +584,11 @@ export const SimulatorCanvas = () => {
const aimX = world.x; const aimX = world.x;
const aimY = world.y + AIMING_OFFSET_Y; const aimY = world.y + AIMING_OFFSET_Y;
setAimPosition({ x: aimX, y: aimY }); setAimPosition({ x: aimX, y: aimY });
const snapDist = AIMING_PIN_SNAP_DIST / zoomRef.current;
const nearPin = findNearestPin(aimX, aimY, snapDist);
setAimHoveredPinName(nearPin ? nearPin.pinName : null);
if (aimPhase === 'aiming_end') { if (aimPhase === 'aiming_end') {
useSimulatorStore.getState().updateWireInProgress(aimX, aimY); useSimulatorStore.getState().updateWireInProgress(aimX, aimY);
} }
@ -687,7 +704,7 @@ export const SimulatorCanvas = () => {
const snapDist = AIMING_PIN_SNAP_DIST / zoomRef.current; const snapDist = AIMING_PIN_SNAP_DIST / zoomRef.current;
const nearPin = findNearestPin(aimX, aimY, snapDist); const nearPin = findNearestPin(aimX, aimY, snapDist);
setWireAiming(false); setAimPosition(null); setWireAiming(false); setAimPosition(null); setAimHoveredPinName(null);
if (nearPin) { if (nearPin) {
// Start wire from this pin // Start wire from this pin
startWireCreation( startWireCreation(
@ -711,7 +728,7 @@ export const SimulatorCanvas = () => {
const snapDist = AIMING_PIN_SNAP_DIST / zoomRef.current; const snapDist = AIMING_PIN_SNAP_DIST / zoomRef.current;
const nearPin = findNearestPin(aimX, aimY, snapDist); const nearPin = findNearestPin(aimX, aimY, snapDist);
setWireAiming(false); setAimPosition(null); setWireAiming(false); setAimPosition(null); setAimHoveredPinName(null);
if (nearPin) { if (nearPin) {
// Finish wire at this pin // Finish wire at this pin
finishWireCreation({ componentId: nearPin.componentId, pinName: nearPin.pinName, x: nearPin.x, y: nearPin.y }); finishWireCreation({ componentId: nearPin.componentId, pinName: nearPin.pinName, x: nearPin.x, y: nearPin.y });
@ -1723,6 +1740,27 @@ export const SimulatorCanvas = () => {
<circle cx={aimPosition.x} cy={aimPosition.y} r="2" fill="rgba(0, 200, 255, 0.9)" /> <circle cx={aimPosition.x} cy={aimPosition.y} r="2" fill="rgba(0, 200, 255, 0.9)" />
</svg> </svg>
)} )}
{/* Aim Hover Tooltip for Touch Devices */}
{wireAiming && aimPosition && aimHoveredPinName && (
<div style={{
position: 'absolute',
left: aimPosition.x + 15,
top: aimPosition.y - 20,
backgroundColor: 'rgba(0, 0, 0, 0.8)',
color: 'white',
padding: '4px 8px',
borderRadius: '4px',
fontSize: '12px',
pointerEvents: 'none',
zIndex: 45,
whiteSpace: 'nowrap',
boxShadow: '0 2px 4px rgba(0,0,0,0.3)',
border: '1px solid rgba(255,255,255,0.1)'
}}>
{aimHoveredPinName}
</div>
)}
</div> </div>
{/* Wire creation mode banner — visible on both desktop and mobile */} {/* Wire creation mode banner — visible on both desktop and mobile */}

View File

@ -55,7 +55,10 @@ export const WireLayer: React.FC<WireLayerProps> = ({
zIndex: 35, zIndex: 35,
}} }}
> >
{wires.map((wire) => ( {wires.map((wire) => {
// Skip null/undefined wires (can happen during circuit loading)
if (!wire) return null;
return (
<WireRenderer <WireRenderer
key={wire.id} key={wire.id}
wire={wire} wire={wire}
@ -67,7 +70,8 @@ export const WireLayer: React.FC<WireLayerProps> = ({
: undefined : undefined
} }
/> />
))} );
})}
{/* Segment handles for the selected wire — scaled inversely to zoom for constant screen size */} {/* Segment handles for the selected wire — scaled inversely to zoom for constant screen size */}
{segmentHandles.map((handle) => { {segmentHandles.map((handle) => {

View File

@ -24,6 +24,9 @@ export const WireRenderer: React.FC<WireRendererProps> = ({
previewWaypoints, previewWaypoints,
overridePath, overridePath,
}) => { }) => {
// Guard against null/undefined wire
if (!wire || !wire.start || !wire.end) return null;
const waypoints = previewWaypoints ?? wire.waypoints; const waypoints = previewWaypoints ?? wire.waypoints;
const path = overridePath ?? generateOrthogonalPath(wire.start, waypoints, wire.end); const path = overridePath ?? generateOrthogonalPath(wire.start, waypoints, wire.end);

View File

@ -92,7 +92,9 @@ class EmbedBridge {
// Set wires if provided // Set wires if provided
if (data.wires && Array.isArray(data.wires)) { if (data.wires && Array.isArray(data.wires)) {
store.setWires(data.wires as never[]); // Filter out null/undefined wires to prevent runtime errors
const validWires = data.wires.filter((w) => w && w.start && w.end && w.id) as never[];
store.setWires(validWires);
} }
break; break;
} }