Skip to content

Stop the movement map opening blurry - #115

Merged
ryanbarlow97 merged 1 commit into
mainfrom
movement-first-view-sharp
Oct 8, 2026
Merged

ryanbarlow97 merged 1 commit into
mainfrom
movement-first-view-sharp

Conversation

@ryanbarlow97

Copy link
Copy Markdown
Contributor

Problem

Opening Staff panel → Movement showed a blurry map until you zoomed in a little; zooming back out was then sharp.

Cause

The map opens on the whole world, then focusMapRect animated a 5×+ zoom (450 ms CSS transform transition) to frame the players. The sharp tiles for the final zoom load and decode while the layer is still drawn small mid-animation, and the browser keeps that low-resolution copy. The next zoom re-lays the layer out at its real size, which is why a small zoom fixes it.

Fix

  • focusMapRect takes an optional { animated } (same shape as resetViewport's).
  • The movement map's first framing, and the Rail tab's (same pattern), jump straight there. Later framings (new session, range, picked line) still animate.
  • Test: first framing is not animated, the next one is.

Headless Chromium/WebKit can't show compositor blur (they repaint everything for screenshots), so this needs checking on a real browser. Locally the first view lands on the same framing and tile level as before, without the animation.

🤖 Generated with Claude Code

The movement and rail maps opened on the whole map and then animated a
zoom of several times to frame the players or network. The sharp tiles
loaded while the layer was still drawn small mid-zoom, and the browser
kept that low-resolution copy until the next zoom. The first framing
now jumps straight there; later framings still animate.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 8, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 57 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e2e9fe36-984d-40e9-a4e0-c9d3742f3d73
📥 Commits

Reviewing files that changed from the base of the PR and between 811196d and 7221340.

📒 Files selected for processing (4)
  • frontend/app/components/admin/MovementMap.test.tsx
  • frontend/app/components/admin/MovementMap.tsx
  • frontend/app/components/admin/RailMap.tsx
  • frontend/app/hooks/useMapViewport.ts
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ryanbarlow97
ryanbarlow97 merged commit 39cf158 into main Oct 8, 2026
2 checks passed
@ryanbarlow97
ryanbarlow97 deleted the movement-first-view-sharp branch October 8, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant