-
Notifications
You must be signed in to change notification settings - Fork 819
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
Label's position sometimes doesn't follow label node's position #1386
Comments
This is a deliberate change, see #1134. |
See issue #105, where @gravitystorm why he doesn't want to use label nodes for rendering. |
Thanks for the links! =} I guess it could be open (at least for now), because I have some further comments and questions. Maybe the subject will deserve renaming (if that's technically possible).
Ok, I see - but we use them at some other levels. So why are we so inconsistent with such "entirely wrong concept"? With such a strong statement we definitely shouldn't.
That would be even better for me than manual labeling! The real problem for me is that sometimes admin_centre name is obscured by admin_level name, which I consider a serious bug for a general map. For example Warsaw is not visible up to z=7 (I think there are some other bugs at low zoom levels, but 6 and 7 seem like obvious cases of above). IMO admin_level name should always try hard to allow admin_centre name to appear where it really is, because it is probably huge and we have more flexibility in choosing the place to render its name. |
I think that it would be better to open ticket for this one (assuming there is not one). Label nodes are resulting in optimization for the most common data consumers what is a rather poor solution. |
You're right. I will open new ticket for improving the renderer, but this ticket should be probably left open for discussion about the first issue - namely why sometimes we use label nodes and whether we should drop using it entirely? |
I don't know - they are long and hairy... =/ Could you recap current state of things regarding label nodes (if you follow the topic, of course)? |
Very briefly summarized:
|
OK, so does it mean @gravitystorm changed his mind since #105 (softened it a little) - and also what was the reasons, in a nutshell? |
I'm not sure if my opinion entirely corresponds to @gravitystorm's opinion, but my main problem is that a label is not a geographic feature and therefore should not be in the database. If the label should be rendered on the centroid, then the renderer is able to compute that without needing a label node. If the label should be rendered out of the way of other features and labels, a label node won't help because this depend on zoomlevel and rendering rules. And if the label should be rendered on the economic centre of a town, then the node should be called place=economic_centre or something like that. |
You're probably right - label node is cheating. =} But my question is why is not how it should be done, but why it's still used then? |
I don't think we use label nodes. We do use place=city nodes, which is also not very elegant, but slightly less hack-ish than label nodes. |
I see. So we can close that ticket now. |
For admin_level=4 label seems to be always rendered in the center of the area:
http://www.openstreetmap.org/relation/130935
while I guess it should follow label node's position:
http://www.openstreetmap.org/node/504977488
The text was updated successfully, but these errors were encountered: