Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#9346 ensure the zoom is always round before using it as index #9357

Merged
merged 1 commit into from
Sep 5, 2023

Conversation

allyoucanmap
Copy link
Contributor

Description

Sometimes the zoom level is to round also inside 2D map, with this PR we ensure the zoom is round when it is used as array index. The Print plugin was using reprojectZoom that was failing when the zoom level had decimals

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix

Issue

What is the current behavior?

#9346

What is the new behavior?

Print plugin is working also after zoom in or out

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • No

Other useful information

see #9346 (comment)

@offtherailz offtherailz merged commit 0d9c33f into geosolutions-it:master Sep 5, 2023
@ElenaGallo ElenaGallo added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Sep 6, 2023
@ElenaGallo
Copy link
Contributor

Test passed, @allyoucanmap please backport this to stable branch. Thanks

allyoucanmap added a commit to allyoucanmap/MapStore2 that referenced this pull request Sep 13, 2023
@allyoucanmap allyoucanmap removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Sep 13, 2023
dsuren1 added a commit to dsuren1/MapStore2 that referenced this pull request Sep 20, 2023
…notation

* commit 'bed88971c58ab68dbfb33183eacaef8481fd0954':
  Adding clause in documentation for google photorealistic 3D tiles (geosolutions-it#9458) (geosolutions-it#9460)
  geosolutions-it#9275 Fix issue Invalid date on Firefox  (geosolutions-it#9393) (geosolutions-it#9451)
  geosolutions-it#9163 Bug 3D Tiles offset with Google 3D tiles (geosolutions-it#9421) (geosolutions-it#9450)
  geosolutions-it#9269 fix issue of correct map embed url in sharing map (geosolutions-it#9395) (geosolutions-it#9452)
  geosolutions-it#9310 Annotation not works with sideBarmenu (geosolutions-it#9402) (geosolutions-it#9449)
  set resolve.fallback.http/https/zlib to false in webpack extension build config (geosolutions-it#9375) (geosolutions-it#9445)
  geosolutions-it#9422 Zoom level issue on OpenLayers (geosolutions-it#9429) (geosolutions-it#9442)
  geosolutions-it#9362 3D Styling issue (geosolutions-it#9403) (geosolutions-it#9441)
  geosolutions-it#9363 Wrong scale value reported using the scroll wheel (geosolutions-it#9401) (geosolutions-it#9440)
  geosolutions-it#9366 Problems with Bearing measurement (geosolutions-it#9400) (geosolutions-it#9439)
  geosolutions-it#9346 ensure the zoom is always round before using it as index (geosolutions-it#9357) (geosolutions-it#9438)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Print plugin is not showing the printable area
3 participants