Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

larger text halo radius for school, college, university & kindergarten #18

Closed
jburgess777 opened this Issue Dec 17, 2012 · 2 comments

Comments

Projects
None yet
3 participants

In the carto style the text radius for school, college, etc is 2, whereas the osm.xml is only 1

osm.xml:
[amenity] = 'school' or [amenity] = 'college'
TextSymbolizer size="9" fill="#33" fontset-name="book-fonts" halo-radius="1" wrap-width="14" placement="interior">[name]

carto:
([amenity] = 'school')
TextSymbolizer size="9" fill="#33" fontset-name="fontset-0" halo-radius="2" wrap-width="14" placement="interior" ><![CDATA[[name]]]>

I can confirm this. I admit though that the halo-radius of 2 makes the label slightly easier to read and would vote to keep the halo-radius at 2.

That said, any style changes weren't going to be done on until v3.0, so just leave this alone until then ? revert it back to 1 now ? Either way, this is a very trivial fix (in changing code). what do you think @gravitystorm ?

Owner

gravitystorm commented Mar 13, 2013

I've reverted it - no particular opinion on which was better, so went with the original.

For what it's worth, I think most of the text is too small, and it'll be better to improve legibility with better fonts, and bigger sizes, rather than just big white halos.

@k127 k127 pushed a commit to k127/openstreetmap-carto that referenced this issue Oct 16, 2016

@giggls giggls Merge pull request #18 from rory/patch-1
Update INSTALL-de.md
a712813
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment