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

Are "identity" zoom functions allowed? #4151

Closed
lucaswoj opened this issue Feb 1, 2017 · 4 comments
Closed

Are "identity" zoom functions allowed? #4151

lucaswoj opened this issue Feb 1, 2017 · 4 comments
Assignees
Labels
cross-platform 📺 Requires coordination with Mapbox GL Native (style specification, rendering tests, etc.)

Comments

@lucaswoj
Copy link
Contributor

lucaswoj commented Feb 1, 2017

From @jfirebaugh on November 30, 2016 2:7

    {
      "id": "fill",
      "type": "fill",
      "source": "source",
      "source-layer": "layer",
      "paint": {
        "fill-opacity": {
          "type": "identity"
        }
      }
    }

Is this allowed?

Copied from original issue: mapbox/mapbox-gl-style-spec#603

@lucaswoj
Copy link
Contributor Author

lucaswoj commented Feb 1, 2017

I can't think of a reason why we shouldn't allow this unless it adds implementation complexity.

@lucaswoj
Copy link
Contributor Author

lucaswoj commented Feb 1, 2017

From @jfirebaugh on November 30, 2016 19:13

It will add four additional rows to #604, which translates to implementation complexity for gl-native.

@jfirebaugh jfirebaugh self-assigned this Feb 3, 2017
@1ec5 1ec5 added the cross-platform 📺 Requires coordination with Mapbox GL Native (style specification, rendering tests, etc.) label Feb 3, 2017
@jplante
Copy link

jplante commented Jul 5, 2017

Unless there is another way to handle circle-radius as a distance, we would like this feature:

For example - ALA Leaflet's Circle and CircleMarker - where the radius is based on either distance or pixels - we could use this for setting different radius for a Circle based on data identity attributes

@jplante
Copy link

jplante commented Jul 5, 2017

According to this, there is no other way to handle circle-radius as a distance:
#2525

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-platform 📺 Requires coordination with Mapbox GL Native (style specification, rendering tests, etc.)
Projects
None yet
Development

No branches or pull requests

4 participants