Apply text-keep-upright after text-offset

[andrewharvey]

Platform: Mapbox Raster Tiles API?
Mapbox SDK version:

equivalent of mapbox/mapbox-gl-js#2350

[jfirebaugh]

@ansis @ChrisLoer Was this included in recent text rendering PRs?

[ChrisLoer]

@jfirebaugh Yeah, it should be handled here:

https://github.com/mapbox/mapbox-gl-native/blob/master/src/mbgl/layout/symbol_projection.cpp#L200

I tested the JS version with @andrewharvey 's jsbin: http://jsbin.com/danexuj/1/edit?html,output. The actual text-offset now works "as expected" (i.e. applied relative to line, not based on text direction). But the original example was actually using text-anchor: bottom instead of text-offset, which still behaves the same as before. Should it? It seems to me that it makes more sense for text-anchor to be based off of text direction.

[ChrisLoer]

PR to enable the test on native: #9580

[stale]

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