Remove miter-limit and round-limit?

[nickidlugash]

Per random conversation with @ansis and @kkaefer: should we remove these two properties for v8? Has anyone actually ever used them? In theory these are useful, but in practice, the default values work well enough that I've never, ever changed them.

@samanpwbb @peterqliu @andreasviglakis have you ever come across joints in geometries that are extreme enough to want to adjust these properties?

/cc @mayagao

[peterqliu]

I haven't changed them, either, but since they're already implemented and there may be future use cases, I'm leaning toward keeping them in.

[ansis]

I'm in favour of removing them.

round-limit isn't a concept that exists elsewhere and there shouldn't be a case where a user needs to change it. We should definitely remove this.

miter-limit already has a hardcoded maximum value because our implementation can't support really long miters. The bevel line join type is what a low miter-limit would look like. I think the difference between what you could do with and without it is tiny, so we should remove it.

[nickidlugash]

round-limit isn't a concept that exists elsewhere

y, I've never seen/used this elsewhere.

I spent some time playing with this today, styling Streets landuse as lines (since those geometries have a lot of irregular edges) with line-widths of various thicknesses over about 5px. Although there are obvious visual differences with using different miter-limit values, I think the default (x2, which is fairly small), is good.

Given that in the last 9 months, no one has noticed that the bevel and miter join icons are currently switched in the app, I don't think anyone will miss miter-limit.

If we want to leave it in though, we could make this property less prominent in the app. Right now it takes up too much space. /cc @mayagao

[samanpwbb]

+1 for removing these!

[nickidlugash]

Anyone else have thoughts on this? Can we remove these for the next style spec version?

[lucaswoj]

This issue was moved to mapbox/mapbox-gl-js#4092

[lucaswoj]

This issue was moved to mapbox/mapbox-gl-js#4103