load_map should warn about missing fonts in fontsets #1791

Closed
herm opened this Issue Apr 2, 2013 · 3 comments

Comments

Projects
None yet
3 participants
@herm
Member

herm commented Apr 2, 2013

Even when strict mode is disabled a missing font in a fontset should display a warning message.

@springmeyer

This comment has been minimized.

Show comment
Hide comment
@springmeyer

springmeyer May 29, 2013

Member

+1 - or even better throw. It looks like this needs to be done in src/text_properties / char_properties::from_xml, but its hard to tell exactly. Maybe checking code should be added to load_map.cpp to avoid needed to repeat in multiple places...

Member

springmeyer commented May 29, 2013

+1 - or even better throw. It looks like this needs to be done in src/text_properties / char_properties::from_xml, but its hard to tell exactly. Maybe checking code should be added to load_map.cpp to avoid needed to repeat in multiple places...

@strk

This comment has been minimized.

Show comment
Hide comment
@strk

strk Nov 12, 2013

Contributor

I'd like to get an error at load time too, as per CartoDB/Windshaft-cartodb#90.
Or (optionally with an XML parameter?) make the missing-face event tolerated by using a different font (system default or something).

Contributor

strk commented Nov 12, 2013

I'd like to get an error at load time too, as per CartoDB/Windshaft-cartodb#90.
Or (optionally with an XML parameter?) make the missing-face event tolerated by using a different font (system default or something).

@strk strk referenced this issue in mapbox/carto Nov 12, 2013

Closed

Better error for font face-name validation #242

@springmeyer

This comment has been minimized.

Show comment
Hide comment
@springmeyer

springmeyer Nov 13, 2013

Member

@strk mapnik.Map.load/loadSync have an optional options argument that can contain {strict: true} which will trigger more font errors a load time.

Member

springmeyer commented Nov 13, 2013

@strk mapnik.Map.load/loadSync have an optional options argument that can contain {strict: true} which will trigger more font errors a load time.

@springmeyer springmeyer added this to the Mapnik 3.x milestone Sep 6, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment