Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

How to treat island labels on z4–z6? #1448

Closed
sommerluk opened this issue Mar 21, 2015 · 7 comments
Closed

How to treat island labels on z4–z6? #1448

sommerluk opened this issue Mar 21, 2015 · 7 comments

Comments

@sommerluk
Copy link
Collaborator

Island labels start currently at z7.

@imagico had proposed to render it starting from z4. This would render labels on some very big islands, e.g. Spitzbergen.

Currently, for low zoom levels, this feature is part of the layer with the id: "text-poly-low-zoom". This layer renders only z7–z9. So I suppose when we want this, we need a new layer? How would that influence the performance? How is the policy for on low zoom levels? Would this interfer with other things on low zoom levels?

Would it make sense to render place=archipelago also? (But is not supported by osm2pgsql? 134 relations with this attribute.)

@imagico
Copy link
Collaborator

imagico commented Mar 21, 2015

Since many islands at the moment have a label of an administrative unit, often wrong because it refers to a unit the island only belongs to, see for example:

http://www.openstreetmap.org/#map=4/79.07/47.99
http://www.openstreetmap.org/#map=7/30.458/-18.127
http://www.openstreetmap.org/#map=7/-57.880/-28.169

it would make sense to consider these two labeling problems together.

Rendering place=archipelago should be fairly easy in principle. However the label of an archipelago should not be placed on one of the islands but somewhere near its center, possibly even over water.

@imagico
Copy link
Collaborator

imagico commented Jul 22, 2017

I think now that we got rid of the multiple labels for admin multipart admin unit polygons it is much clearer where this would improve the map. Examples where this would be nice to have:

http://www.openstreetmap.org/#map=6/-0.736/110.720
http://www.openstreetmap.org/#map=5/76.614/-100.327

http://www.openstreetmap.org/#map=6/78.864/17.688
http://www.openstreetmap.org/#map=6/79.642/95.713
http://www.openstreetmap.org/#map=6/73.846/57.678
http://www.openstreetmap.org/#map=6/74.894/141.812

http://www.openstreetmap.org/#map=6/-71.403/-72.246

z4 labels would be mostly of interest for archipelagos and less for individual islands (except for Canada and Indonesia). It would be interesting to check if labeling archipelagos using ST_PointOnSurface(ST_ConvexHull(way)) would be all right or if it leads to problem in some cases.

@kocio-pl
Copy link
Collaborator

What if we try the same solution as with water area labels? I mean: start rendering island labels from z0, because we already use the same area limit (>3000). I would relax initial zoom level also for islets - if they are big enough to handle label, just render it there.

@kocio-pl kocio-pl self-assigned this Oct 14, 2017
@kocio-pl
Copy link
Collaborator

OK, z0-z3 does not make sense, because we have no land labels before z3, and on z3 islands would compete with countries, so I support original call for z4+ for islands (Kosmtik export is not working again, so using shiny new Firefox screenshots feature):

Nunavut, Canada

z3
Before
screenshot-2017-10-14 openstreetmap carto kosmtik before
After
screenshot-2017-10-14 openstreetmap carto kosmtik after

z4
Before
screenshot-2017-10-14 openstreetmap carto kosmtik
After
screenshot-2017-10-14 openstreetmap carto kosmtik 1

@kocio-pl
Copy link
Collaborator

Spitzbergen would be not visible anyway, because it's tagged as a node currently:

http://www.openstreetmap.org/node/269078675#map=5/77.934/20.149

but some smaller island around would be from z4:

screenshot-2017-10-14 openstreetmap carto kosmtik 2

@matthijsmelissen
Copy link
Collaborator

+1

@kocio-pl
Copy link
Collaborator

I've just fixed the island relation for Spitsbergen, because it was there already, but not complete, so now it should also render the label:

http://www.openstreetmap.org/relation/1124190

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

No branches or pull requests

5 participants