fix(webapp): the map's thumbnail under its tiles, and a taller map on a phone (#1286) #1287

Merged

claude-bot merged 1 commit from fix/4.6.2-rc4 into main 2026-10-30 08:12:17 +00:00

Conversation 0 Commits 1 Files changed 4 +26 -5

claude-bot commented 2026-10-30 07:19:20 +00:00

Collaborator

Copy link

Closes #1286. For 4.6.2-rc4: the owner's findings on rc3, on a phone.

• The map never sharpened on zoom. #1282's thumbnail L.imageOverlay used Leaflet's overlay pane (z-index 400 in leaflet.css), which sits above the tile pane (200), so it covered the sharp tiles. It now has its own pane, qmapThumb, at 150. The canvas test asserts the pane and its z-index.
• The map's box was a short strip on a phone. The inline aspectRatio becomes --qmap-ratio, still used on wide screens with the existing max-height. At 640 px and under, the box is 70svh tall.

Checked: vitest src/pages/wiki (349 passed); eslint reports 0 errors; changelog.py check passes. Frontend only, no backend change.
Not checked: not seen on a real phone; the owner checks on rc4.

🤖 Generated with Claude Code

Closes #1286. For 4.6.2-rc4: the owner's findings on rc3, on a phone. - **The map never sharpened on zoom.** #1282's thumbnail `L.imageOverlay` used Leaflet's overlay pane (z-index 400 in `leaflet.css`), which sits above the tile pane (200), so it covered the sharp tiles. It now has its own pane, `qmapThumb`, at 150. The canvas test asserts the pane and its z-index. - **The map's box was a short strip on a phone.** The inline `aspectRatio` becomes `--qmap-ratio`, still used on wide screens with the existing max-height. At 640 px and under, the box is `70svh` tall. **Checked:** vitest `src/pages/wiki` (349 passed); eslint reports 0 errors; `changelog.py check` passes. Frontend only, no backend change. **Not checked:** not seen on a real phone; the owner checks on rc4. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude-bot added 1 commit 2026-10-30 07:19:21 +00:00

fix(webapp): the map's thumbnail under its tiles, and a taller map on a phone (#1286) ... 094bf7ad76

The thumbnail overlay was in Leaflet's overlay pane (400), above the tile
pane (200), so the sharp tiles never showed; it gets its own pane at 150.
The canvas's aspect ratio moves to a CSS variable, and at 640 px and under
the box is 70svh tall instead of the map's shape.

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

claude-bot scheduled this pull request to auto merge when all checks succeed 2026-10-30 07:19:30 +00:00

claude-bot merged commit 24fce5f3c4 into main 2026-10-30 08:12:17 +00:00

claude-bot deleted branch fix/4.6.2-rc4 2026-10-30 08:12:18 +00:00

claude-bot referenced this pull request from a commit 2026-10-30 08:12:19 +00:00

Merge pull request 'fix(webapp): the map's thumbnail under its tiles, and a taller map on a phone (#1286)' (#1287) from fix/4.6.2-rc4 into main

claude-bot referenced this pull request 2026-10-30 08:12:42 +00:00

chore(release): v4.6.2-rc4 #1288

claude-bot referenced this pull request 2026-10-30 18:32:59 +00:00

Map: what the plan has that the build lacks (audit, 2026-10-28) #1263

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

backend

Issue related to the backend

  

bot

Issue related to the bot

  

bug

Something is not working

  

duplicate

This issue or pull request already exists

  

enhancement

New feature

  

frontend

Issue related to the frontend

  

help wanted

Need some help

  

invalid

Something is wrong

  

question

More information is needed

  

recording

Issue related to the recording system

  

security

Security hardening / vulnerability work

  

wontfix

This won't be fixed

No labels

backend

bot

bug

duplicate

enhancement

frontend

help wanted

invalid

question

recording

security

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

rbrooks/Quest-Board!1287

No description provided.