-
-
Notifications
You must be signed in to change notification settings - Fork 67
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
Design proposal: Add elevation to symbol layer #62
Comments
TSC meeting on April 12 notes: Action points:
|
For MapComplete, this would be really great to have - e.g. if a POI is located at a higher floor (or to do indoor maps). I see two approaches for this:
My first instinct is that using the coordinate from the geojson would be the most versatile (but I have not experimented with this) |
Another potentially use for this is using a I think limiting this to only GeoJSON sources would be too limiting. |
In maplibre-gl-js geojson is converted to mvt which doesn't have elevation, so this proposal is less about adding elevation to geometries and more about adding a field in the spec |
Having an attribute would work for points, but it wouldn't for lines. Thus it would be better to support 3D coordinates:
As mapbox currently seems to ignore all 3D point/line issues (e.g. mapbox/vector-tile-spec#111) this could be a major feature of maplibre. |
We are actively working on the next vector tiles format, but it will take time... |
Wow didn't know about these, super impressive. Google wasn't working for me but I did eventually find the repos: https://github.com/mactrem/com-tiles |
This still needs a bit more refinement, but the general idea is that a symbol layer should be allowed to be presented above the ground, this isn't only for when terrain is on but should also be applied when extrusion is shows to show markers, icons, and text above the ground.
The original request was described here:
maplibre/maplibre-gl-js#644
Since this is somewhat related to 3D visualization I think it can be linked to the following bounty:
maplibre/maplibre#189
Once this is refined and approved we can split this into specific issues and assigned bounties.
The text was updated successfully, but these errors were encountered: