tekki/nissefolk
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Issue #5: Mouse handling — zoom-to-mouse + middle-click pan #10

Merged
tekki merged 21 commits from feature/mouse-handling into master 2026-03-21 14:06:45 +00:00
Conversation 1 Commits 21 Files Changed 8 +871 -10
claude commented 2026-03-20 19:23:32 +00:00
Collaborator
Copy Link

Closes #5

Was sich geändert hat

CameraSystem.ts

  • Zoom-to-Mouse: Nach cam.setZoom() wird der Scroll korrigiert, sodass der Weltpunkt unter dem Cursor fixiert bleibt
  • Formel: scrollX += (mouseX - screenW/2) * (1/zoomBefore - 1/zoomAfter)
  • getCenterWorld() korrigiert: war scrollX + width/(2·zoom), korrekt ist scrollX + width/2 (Phaser zoomt vom Screenzentrumm, nicht von der oberen linken Ecke)
  • JSDoc auf allen Methoden ergänzt
  • Middle-click pan unverändert beibehalten

src/test/ — Test-Infrastruktur (bleibt im Repo)

  • test.html + src/test/main.ts: eigener Vite-Entry, komplett getrennt vom Spiel
  • ZoomTestScene: Phaser-Default-Zoom zum Beobachten des Ankerpunkt-Verhaltens
  • ZoomMouseScene: Zoom-to-Mouse zum direkten Vergleich; Tab wechselt zwischen beiden
  • Vite-Middleware (/api/log): schreibt JSON-Logs nach game-test.log für Offline-Analyse

Hintergrund

Der korrekte Zoom-Ankerpunkt von Phaser war nicht dokumentiert und ließ sich nur empirisch über die Test-Szenen ermitteln. Die Ergebnisse sind in Issue #5 kommentiert.

Test

  • Scrollrad an verschiedenen Screenpositionen: Weltpunkt unter Cursor bleibt fixiert
  • Scrollrad in Screenmitte: verhält sich wie vorher (Zoom zur Mitte)
  • Mittelklick halten + ziehen: Camera-Pan
  • WASD/Pfeiltasten: unverändert
  • Zoom an Weltgrenzen: korrekt geclampt
Closes #5 ## Was sich geändert hat ### `CameraSystem.ts` - **Zoom-to-Mouse**: Nach `cam.setZoom()` wird der Scroll korrigiert, sodass der Weltpunkt unter dem Cursor fixiert bleibt - **Formel**: `scrollX += (mouseX - screenW/2) * (1/zoomBefore - 1/zoomAfter)` - **`getCenterWorld()`** korrigiert: war `scrollX + width/(2·zoom)`, korrekt ist `scrollX + width/2` (Phaser zoomt vom Screenzentrumm, nicht von der oberen linken Ecke) - **JSDoc** auf allen Methoden ergänzt - Middle-click pan unverändert beibehalten ### `src/test/` — Test-Infrastruktur (bleibt im Repo) - `test.html` + `src/test/main.ts`: eigener Vite-Entry, komplett getrennt vom Spiel - `ZoomTestScene`: Phaser-Default-Zoom zum Beobachten des Ankerpunkt-Verhaltens - `ZoomMouseScene`: Zoom-to-Mouse zum direkten Vergleich; Tab wechselt zwischen beiden - Vite-Middleware (`/api/log`): schreibt JSON-Logs nach `game-test.log` für Offline-Analyse ### Hintergrund Der korrekte Zoom-Ankerpunkt von Phaser war nicht dokumentiert und ließ sich nur empirisch über die Test-Szenen ermitteln. Die Ergebnisse sind in Issue #5 kommentiert. ## Test - Scrollrad an verschiedenen Screenpositionen: Weltpunkt unter Cursor bleibt fixiert - Scrollrad in Screenmitte: verhält sich wie vorher (Zoom zur Mitte) - Mittelklick halten + ziehen: Camera-Pan - WASD/Pfeiltasten: unverändert - Zoom an Weltgrenzen: korrekt geclampt
claude added 2 commits 2026-03-20 19:23:32 +00:00
✨ zoom to mouse pointer + middle-click pan 715278ae78 
- Scroll wheel now zooms toward the mouse cursor instead of screen center
- Middle mouse button held: pan camera by dragging
- Both actions respect current zoom level
📝 update CHANGELOG for Issue #5 mouse handling fa41075c55
claude added 1 commit 2026-03-20 19:29:57 +00:00
🐛 fix zoom-to-mouse using getWorldPoint diff instead of manual formula f0065a0cda
claude added 1 commit 2026-03-20 19:39:16 +00:00
🐛 fix zoom: center world point under mouse, then zoom to center b5130169bd
claude added 1 commit 2026-03-20 20:15:16 +00:00
🐛 zoom-to-mouse: use ptr.worldX/Y + set scroll after setZoom 216c70dbd9
claude added 1 commit 2026-03-20 20:21:52 +00:00
♻️ revert zoom to simple center zoom, remove mouse targeting cccfd9ba73
claude added 1 commit 2026-03-20 20:34:35 +00:00
🐛 debug: log mouse+center on zoom, draw red cross at viewport center 0e4c7c96ee
claude added 1 commit 2026-03-20 20:37:12 +00:00
🐛 zoom-to-mouse: ptr.worldX/Y formula, debug log still active fb4abb7256
claude added 1 commit 2026-03-20 20:39:55 +00:00
🐛 fix zoom-to-mouse: capture worldX/Y before setZoom d354a26a80
claude added 1 commit 2026-03-20 20:45:21 +00:00
🐛 zoom-to-mouse: track world coords on pointermove, avoid ptr.worldX getter 6de4c1cbb9
claude added 1 commit 2026-03-20 20:57:48 +00:00
🐛 fix debug cross: world-space position + counter-scale, tracks viewport center correctly 6fa3ae4465
claude added 1 commit 2026-03-20 21:06:15 +00:00
🐛 fix debug cross: clear+redraw each frame at world-space center, no transforms 0011bc9877
claude added 1 commit 2026-03-20 21:09:17 +00:00
♻️ revert zoom to center-only, keep middle-click pan 34220818b0
claude added 6 commits 2026-03-21 12:00:28 +00:00
✨ add ZoomTestScene with Phaser default zoom for analysis 007d5b3fee 
Separate test environment at /test.html (own Vite entry, own Phaser
instance). ZoomTestScene renders a 50×50 tile grid with crosshair
markers and a live HUD overlay showing zoom, scroll, viewport in px
and tiles, mouse world/screen/tile coords, and renderer info.
Zoom uses plain cam.setZoom() — no mouse tracking — to observe
Phaser's default center-anchor behavior.
✨ add file logging via Vite middleware to ZoomTestScene 9b6341fe46 
Vite dev server gets a /api/log middleware (POST appends to
game-test.log, DELETE clears it). ZoomTestScene writes a zoom event
with before/after state on every scroll, plus a full snapshot every
2 seconds. Log entries are newline-delimited JSON.
🐛 fix HUD overlay zoom + add red center crosshair faa4deb0bf 
Text overlay now uses a dedicated HUD camera (zoom=1, fixed scroll)
so it's never scaled by the world zoom. World objects and HUD objects
are separated via camera ignore lists. Added red screen-center
crosshair to HUD layer as a precise alignment reference.
♻️ increase test world to 500×500 tiles, adjust marker intervals aefb67dba6
✨ add ZoomMouseScene with zoom-to-mouse correction fede13d64a 
Implements scroll correction after cam.setZoom() so the world point
under the mouse stays fixed. Formula accounts for Phaser's
center-based zoom: scrollX += (mouseX - cw/2) * (1/zBefore - 1/zAfter).
Tab switches between the two test scenes in both directions.
Also fixes centerWorld formula in ZoomTestScene overlay and logs.
✨ implement zoom-to-mouse in CameraSystem 0f411f0f34 
Replaces plain cam.setZoom() with zoom-to-mouse: after each zoom step
the scroll is corrected by (mouseOffset from center) * (1/zBefore - 1/zAfter),
keeping the world point under the cursor fixed. Also fixes getCenterWorld()
which previously divided by zoom incorrectly. Added JSDoc to all methods.
claude added 1 commit 2026-03-21 12:01:41 +00:00
📝 update CHANGELOG for Issue #5 zoom-to-mouse 49fae62f27
claude changed title from Issue #5: Mouse handling — zoom to cursor + middle-click pan to Issue #5: Mouse handling — zoom-to-mouse + middle-click pan 2026-03-21 12:04:51 +00:00
claude added 2 commits 2026-03-21 12:36:24 +00:00
📝 update CHANGELOG for Issue #5 zoom-to-mouse 49fae62f27
🔒 ignore .claude/ dir and game-test.log 1ba38cc23e
tekki approved these changes 2026-03-21 14:04:32 +00:00
tekki merged commit 4c41dc9205 into master 2026-03-21 14:06:45 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
tekki
Labels
Clear labels
bug
design
feature
improvement
question
status: backlog
status: done
status: in discussion
status: in progress
status: ready
status: review
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claude tekki
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tekki/nissefolk#10
No description provided.
Delete Branch "feature/mouse-handling"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?