geoadmin / web-mapviewer

The viewer application for maps and geodata
https://map.geo.admin.ch
Other
57 stars 9 forks source link

PB-395: enhancing highest zoom level - #patch #997

Closed ltkum closed 2 months ago

ltkum commented 2 months ago

Issue : At the highest zoom level, the mapviewer would not load the highest possible resolution, but the one right before it. Cause : When we introduced the max resolution to handle client side zoom, we made a slight error in the slicing, and it caused the possible resolutions client side to be smaller (by 1). Fix : We slice at the correct place. Bonus Fix : when trying to print at the 1:500 and 1:1000 scales, since the matrixes were sent as parameter by the viewer, we now request the correct tiles. I will deploy the print with the updated capabilities to int once both PR are merged and when we are confident this works, we can deploy the new print to prod.

Test link

cypress[bot] commented 2 months ago

Passing run #2949 ↗︎

0 211 21 0 Flakiness 0

Details:

Merge branch 'master' into bugfix-PB-395_print-1_500
Project: web-mapviewer Commit: 8410fa620d
Status: Passed Duration: 04:59 💡
Started: Jul 8, 2024 2:32 PM Ended: Jul 8, 2024 2:37 PM

Review all test suite changes for PR #997 ↗︎