Skip to content

fix: call resize() on network map when sidebar mounts in desktop layout - #66

Merged
mitoperni merged 1 commit into
mainfrom
fix/map-resize-networks-sidebar
Feb 19, 2026
Merged

fix: call resize() on network map when sidebar mounts in desktop layout#66
mitoperni merged 1 commit into
mainfrom
fix/map-resize-networks-sidebar

Conversation

@mitoperni

@mitoperni mitoperni commented Feb 19, 2026

Copy link
Copy Markdown
Owner

Description

Same root cause as #65: on desktop the map container starts as absolute inset-0 (full screen) and transitions to relative flex-1 (excluding sidebar) after React hydrates. The network map was affected too, causing cluster zoom interactions to be slightly off-center.

Type of Change

  • Bug fix

Changes Made

  • Add useEffect in MapboxMap that calls resize() when hasMounted transitions to true, so Mapbox recalculates the container dimensions after the sidebar occupies its real space

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • No TypeScript errors
  • Tests added/updated
  • All CI checks pass
  • Tested on different screen sizes
  • Loading states implemented
  • Error handling in place

Related Issues

Related to #65


Open with Devin

@vercel

vercel Bot commented Feb 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cyclemap Ready Ready Preview, Comment Feb 19, 2026 8:43am

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@mitoperni
mitoperni merged commit f77a889 into main Feb 19, 2026
6 checks passed
@mitoperni
mitoperni deleted the fix/map-resize-networks-sidebar branch February 19, 2026 08:46
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