jan-henrik/respira
1
0
Fork 0
mirror of https://github.com/jhbruhn/respira.git synced 2026-01-27 10:23:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: Correct center pattern button calculation for Konva pivot points

Browse source 
The center pattern function was using the old calculation that didn't account for Konva's offsetX/offsetY pivot points. Since the pattern's center is now its pivot point, centering at the origin is simply {x: 0, y: 0}.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
Jan-Henrik Bruhn 2025-12-25 21:49:33 +01:00
parent a97439da7b
commit 271229c200
1 changed files with 5 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
src/hooks/usePatternTransform.ts
View file

@ -46,10 +46,6 @@ export function usePatternTransform({
localPatternOffset.y !== initialPatternOffset.y) localPatternOffset.y !== initialPatternOffset.y)
) { ) {
setLocalPatternOffset(initialPatternOffset); setLocalPatternOffset(initialPatternOffset);
console.log(
"[PatternTransform] Restored pattern offset:",
initialPatternOffset,
);
} }
// Update pattern rotation when initialPatternRotation changes // Update pattern rotation when initialPatternRotation changes
@ -63,22 +59,13 @@ export function usePatternTransform({
// Attach/detach transformer based on state // Attach/detach transformer based on state
const attachTransformer = useCallback(() => { const attachTransformer = useCallback(() => {
if (!transformerRef.current || !patternGroupRef.current) { if (!transformerRef.current || !patternGroupRef.current) {
console.log(
"[PatternTransform] Cannot attach transformer - refs not ready",
{
hasTransformer: !!transformerRef.current,
hasPatternGroup: !!patternGroupRef.current,
},
);
return; return;
} }
if (!patternUploaded && !isUploading) { if (!patternUploaded && !isUploading) {
console.log("[PatternTransform] Attaching transformer");
transformerRef.current.nodes([patternGroupRef.current]); transformerRef.current.nodes([patternGroupRef.current]);
transformerRef.current.getLayer()?.batchDraw(); transformerRef.current.getLayer()?.batchDraw();
} else { } else {
console.log("[PatternTransform] Detaching transformer");
transformerRef.current.nodes([]); transformerRef.current.nodes([]);
} }
}, [patternUploaded, isUploading]); }, [patternUploaded, isUploading]);
@ -99,12 +86,12 @@ export function usePatternTransform({
const handleCenterPattern = useCallback(() => { const handleCenterPattern = useCallback(() => {
if (!pesData) return; if (!pesData) return;
const { bounds } = pesData; // Since the pattern Group uses offsetX/offsetY to set its pivot point at the pattern center,
const centerOffsetX = -(bounds.minX + bounds.maxX) / 2; // we just need to position it at the origin (0, 0) to center it in the hoop
const centerOffsetY = -(bounds.minY + bounds.maxY) / 2; const centerOffset = { x: 0, y: 0 };
setLocalPatternOffset({ x: centerOffsetX, y: centerOffsetY }); setLocalPatternOffset(centerOffset);
setPatternOffset(centerOffsetX, centerOffsetY); setPatternOffset(centerOffset.x, centerOffset.y);
}, [pesData, setPatternOffset]); }, [pesData, setPatternOffset]);
// Pattern drag handlers // Pattern drag handlers
@ -142,13 +129,6 @@ export function usePatternTransform({
// Store rotation angle and position // Store rotation angle and position
setPatternRotation(normalizedRotation); setPatternRotation(normalizedRotation);
setPatternOffset(newOffset.x, newOffset.y); setPatternOffset(newOffset.x, newOffset.y);
console.log(
"[PatternTransform] Transform end - rotation:",
normalizedRotation,
"degrees, position:",
newOffset,
);
}, },
[setPatternRotation, setPatternOffset, pesData], [setPatternRotation, setPatternOffset, pesData],
); );