get rid of text-max-size and icon-max-size

[ansis]

currently
text-max-size is a layout property and is used for label placement
text-size is a paint property is used to change

We should only have one property.

How would we deal with a property that is need both for layout and rendering?

[jfirebaugh]

layout could calculate what the actual maximum text-size is going to be, right?

[mourner]

Yeah, the only problem is that we introduce a new concept here: a paint property that implicitly affects the layout step as well. But it's probably fine.

[jfirebaugh]

Hrm, yeah, setPaintProperty would then have to special-case text-size to trigger a relayout.

[mourner]

Or we could have it the other way around — make it a layout property that implicitly creates a paint property.

[jfirebaugh]

It would still need to support interpolated functions, which would be a different special case. But maybe a less problematic one.

[nickidlugash]

@jfirebaugh @mourner @ansis any consensus on what we should do here?

[nickidlugash]

I added this to v8 omnibus.

[nickidlugash]

We should also get rid of icon-max-size. I've added that to this ticket name, and will add to v8.

[jfirebaugh]

@ansis, can you look into this when you get back?

[ansis]

pull request for -js: mapbox/mapbox-gl-js#1419

Is the rest of v8 implemented in -native?