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

map-input is not respected for a map-extent and the tiles are still fetched at a lower zoom level #897

Closed
AliyanH opened this issue Oct 31, 2023 · 3 comments
Labels
Milestone

Comments

@AliyanH
Copy link
Member

AliyanH commented Oct 31, 2023

Discovered in #887
Steps to reproduce:

  1. visit the sandbox and paste in the following code
<mapml-viewer projection="CBMTILE" zoom="8" lat="48.840422" lon="-101.219978" controls>
      <layer- label="CBMT" checked>
        <map-extent units="CBMTILE"  checked="checked" hidden="hidden">
          <map-input name="z" type="zoom" min="6" max="8"></map-input>
          <map-input name="y" type="location" units="tilematrix" axis="row"></map-input>
          <map-input name="x" type="location" units="tilematrix" axis="column"></map-input>
          <map-link rel="tile" tref="https://geoappext.nrcan.gc.ca/arcgis/rest/services/BaseMaps/CBMT3978/MapServer/tile/{z}/{y}/{x}?m4h=t" ></map-link>
        </map-extent>
      </layer->
    </mapml-viewer>
  1. Expected behavior - we think the layers should not be fetched outside of zoom level 6-8 (due to the map-input on the map-extent), but they still do when we zoom to the layer through the context menu

Related to #896

@prushforth
Copy link
Member

I had the idea that we might implement "min/maxNativeZoom" by using the min/max attributes of <map-input type=zoom min=3 max=7> for example. Those min/max attributes are as close to the template that loads the tiles as we will get, and can be set up by the map author. The <map-meta name="zoom" content="min=7,max=12"></map-meta> values can control the display of the layer. To be discussed.

@AliyanH AliyanH added this to the Autumn 2023 milestone Nov 8, 2023
@prushforth
Copy link
Member

Possibly related to #666

@AliyanH
Copy link
Member Author

AliyanH commented Aug 20, 2024

This issue has been resolved.

@AliyanH AliyanH closed this as completed Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants