Skip to content
This repository

Add `text-multi-policy` #1583

Open
springmeyer opened this Issue · 5 comments

3 participants

Dane Springmeyer Artem Pavlenko strk
Dane Springmeyer
Owner

We need a text parameter to match the marker-multi-policy added in #1573

strk

I'm testing this but can't figure the current default policy.
Did it change from "centroid" to "interior" while always keeping "whole" as the multi-policy ?

strk

Uhm, I could have found it:
https://github.com/mapnik/mapnik/blob/v2.1.0/src/symbolizer_helpers.cpp#L167

So is it using largest_bbox ? And it looks there is already support for that policy ?

strk

So it looks like Artem added "largest-bbox-only" property on June 2012: 2025596 and exposed it to python binding. It defaults to true but isn't found in the mapnik-reference.

Dane Springmeyer
Owner

Right, I never added it to the mapnik-reference because I was not confident the parameter was 1) named well, or 2) handled all cases. Hence, this ticket. #1573 should be a guide for implementing an equivalently named and functioning parameter.

Dane Springmeyer
Owner

assigning to 3.x, we should dealt with this work potentially as part of the harfbuzz merge.

Dane Springmeyer springmeyer referenced this issue in gravitystorm/openstreetmap-carto
Open

Recommend -G flag for multipolygons #59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.