Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

MGLLineStyleLayer - Gap Line(s) intersect when angled less than 90º #13227

Closed
riastrad opened this issue Oct 29, 2018 · 5 comments
Closed

MGLLineStyleLayer - Gap Line(s) intersect when angled less than 90º #13227

riastrad opened this issue Oct 29, 2018 · 5 comments
Labels
archived Archived because of inactivity bug Core The cross-platform C++ core, aka mbgl design runtime styling

Comments

@riastrad
Copy link
Contributor

riastrad commented Oct 29, 2018

On iOS using MGLLineStyleLayer, when setting the .lineGapWidth attribute, you would expect that the gap line that's generated would not intersect. However, if the Polyline you are styling has angles that are less than 90º these lines intersect.

Steps to reproduce

  1. Create a polyline with angles that < 90º
  2. Style the line using the line-gap-width property
  3. Observe that the line gap intersects

Expected behavior

As this is property is often used to generate an 'outline' of a Polyline feature, it would be expected that the exterior lines would not intersect. Here's how a line gap is rendered on other GIS software:

line-gap

Actual behavior

You get intersecting lines:

iOS Maps SDK
screen shot 2018-10-29 at 4 52 38 pm

Mapbox GL JS
screen shot 2018-10-29 at 5 05 13 pm

Configuration

I've confirmed this behavior in both the iOS Maps SDK v4.5.0 and Mapbox GL JS v0.50.0

@stale
Copy link

stale bot commented Apr 27, 2019

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Apr 27, 2019
@riastrad riastrad reopened this Apr 29, 2019
@stale stale bot removed the archived Archived because of inactivity label Apr 29, 2019
@stale stale bot added the archived Archived because of inactivity label Oct 26, 2019
@stale
Copy link

stale bot commented Oct 26, 2019

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Oct 26, 2019
@chloekraw
Copy link
Contributor

@mourner would this issue be fixed by mapbox/mapbox-gl-js#8275, and was it / will it be ported to -native?

@mourner
Copy link
Member

mourner commented Oct 27, 2019

@chloekraw no, this seems like an unrelated issue.

@stale stale bot removed the archived Archived because of inactivity label Feb 14, 2020
@stale stale bot added the archived Archived because of inactivity label Aug 16, 2020
@stale
Copy link

stale bot commented Aug 16, 2020

This issue has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this as completed Aug 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
archived Archived because of inactivity bug Core The cross-platform C++ core, aka mbgl design runtime styling
Projects
None yet
Development

No branches or pull requests

4 participants