-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
mapbox: Implement Z-index and polyline spans handling #3058
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
base: master
Are you sure you want to change the base?
Conversation
|
A little explanation is probably necessary to understand some of the main choices:
|
Someone asked me over email why this is the case. As I was gathering the info to provide an answer I actually found a potential hacky way to implement it. Maybe next month if I have some time I can play with the idea. This is my response I gave the person, for whoever may be interested:
|
4d20102 to
7c6e79c
Compare
|
@depau I had come to a similar rough conclusion w.r.t. implementing tile overlays, so overall I agree with this approach. Thanks for giving it further thought! Please do it in a separate PR though 😃 I'm intending on reviewing your changes from this PR but alas, time is short. |
|
No rush, it doesn't look like the next CalyxOS release is going to come any time soon so I'll have to make do with black lines on Garmin Connect anyway :) Sure, I'll do that in another PR, it shouldn't too hard to integrate any layer changes with these changes (assuming the overall design doesn't change too much over here) Thank you! |

Hi!
This is a preliminary PoC implementation that adds handling of Z-index and polyline spans to the MapBox map implementation.
It isn't thoroughly tested, though it should fix some of the annoying glitches reported in #2382. Primarily it fixes my annoyance with Garmin Connect not actually displaying the run stats on the map.
I'm not a huge fan of the implementation, I basically did the least amount of work to get the feature going as a proof of concept to make sure this was even possible with the MapLibre API. If you'd like to integrate it I'm open to guidance and suggestions on how to properly implement it.
I also implemented JointType handling.
Before:

After:
Website (just as a reference, I currently don't have a device with GApps to test):
The reason why the colors are slightly different in this image is that I ran multiple times in the same spots. I'll need to compare with the Google Android implementation since I think their web implementation is a bit messy.
EDIT:
Google Maps vs microG:
I think the result is very acceptable.