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.

Sign up for GitHub

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

Jump to bottom

Render place=county/neighbourhood/block #107

Closed
joakimfors opened this issue Aug 8, 2013 · 8 comments
Closed

Render place=county/neighbourhood/block #107

joakimfors opened this issue Aug 8, 2013 · 8 comments
Labels
enhancement new features Requests to render new features placenames
Milestone
New features

Comments

@joakimfors
Copy link

It would be nice if more place types were rendered at appropriate zooms. For example county, neighbourhood, block etc. Both on nodes and closed ways/relations.
@matthijsmelissen matthijsmelissen added this to the New features milestone Aug 18, 2014
@daganzdaanda
Copy link

Yes, see #103 and #546

@dieterdreist
Copy link

Il giorno 06/set/2014, alle ore 23:29, daganzdaanda [email protected] ha scritto:

Yes, see #103

Isn't county an administrative division? OK, country is as well, maybe our tagging should be revised?=

@matthijsmelissen
Copy link
Collaborator

  • place=neighbourhood is already rendered
  • place=county is an administrative region, and the name is already rendered on the admin boundary. I don't think rendering nodes makes much sense, as they are often duplicate with the country capital. I'd propose not to render this.
  • We could render place=block. What is it currently used for?

@matthijsmelissen
Copy link
Collaborator

There is also a place=city_block.

matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Jan 21, 2015
@matthijsmelissen
Clean up place name rendering
c1dcd0d 
* Don't render place=town on z16 and higher, and place=suburb and place=village
  on z17 and higher.
* Render place=hamlet, place=locality, place=neighbourhood, place=quarter,
  place=isolated_dwelling, and place=farm from z15 rather than from z14
  (resolves gravitystorm#948).
* Render place=town black rather than gray.
* Make font sizes more consistent by increasing font size for place=town on z12
  and z13, for place=suburb on z16, and for place=village on z14 and z16
  (resolves gravitystorm#734, resolves gravitystorm#1204).
* Add rendering for place=block and place=city_block (resolves gravitystorm#107).
* Add rendering for place=quarter (like place=neighbourhood) and place=borough
  (like place=suburb, resolves gravitystorm#798).
* Treat capital=4 like capital=yes (resolves gravitystorm#522).
* Make text-wrap consistent.
@matthijsmelissen matthijsmelissen mentioned this issue Jan 21, 2015
Merged
@lest69
Copy link

lest69 commented Jan 22, 2015

We could render place=block. What is it currently used for?

It's unclear, since there's no documentation, but it seems to me to be a duplicate of city_block. Both it and place=city_block appear to be used almost exclusively in Sweden. Oddly, they seem to be used for the same purpose, but are used differently in different cities (ie. city_block in Stockholm, but block in Malmö, Lund, and Landskrona).

Should we be encouraging such ambiguous tagging by rendering both, or only render the documented tag?

@matthijsmelissen
Copy link
Collaborator

Should we be encouraging such ambiguous tagging by rendering both, or only render the documented tag?

In cases like bookmaker/betting and fishmonger/seafood we render both the documented and undocumented (but still used) tag. I don't know if that's a good approach.

Maybe someone could reach out to the Swedish community so they can unify their tags?

@pnorman
Copy link
Collaborator

pnorman commented Jan 22, 2015

Should we be encouraging such ambiguous tagging by rendering both, or only render the documented tag?

If they're both only used in Sweden, I'd rather not render either to encourage usage of a widely used tag.

@matthijsmelissen
Copy link
Collaborator

Rejected per #1247 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement new features Requests to render new features placenames
Projects
None yet
Milestone
New features
Development

No branches or pull requests
6 participants
@dieterdreist @pnorman @joakimfors @matthijsmelissen @lest69 @daganzdaanda