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

include spacial urban amenities in 'built-up-upper-lowzoom' #3007

Open
polarbearing opened this issue Jan 7, 2018 · 7 comments · May be fixed by #3670
Open

include spacial urban amenities in 'built-up-upper-lowzoom' #3007

polarbearing opened this issue Jan 7, 2018 · 7 comments · May be fixed by #3670

Comments

@polarbearing
Copy link
Contributor

polarbearing commented Jan 7, 2018

In mid and low zoom, currently the following landuses are rendered in gray (built-up-upper-lowzoom #c0c0c0, built-up-lower-lowzoom 11-12 #aaaaaa):

  • landuse=residential
  • landuse=retail, amenity=marketplace
  • landuse=industrial
  • landuse=commercial

Spacial amenities (i.e. campuses) and other landuses are often left without surrounding landuse tag, thus they leave holes in the midzoom areas that would be considered as built-up.
Examples (currently rendered from):

To avoid the holes, these features should be assigned the built-up colour in the mid and zoom levels.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Jan 7, 2018

It's a follow up to #2654 (in practice that's just fine tuning this code).

Examples of the problem:

@kocio-pl kocio-pl added this to the Bugs and improvements milestone Jan 7, 2018
@drmx
Copy link

drmx commented Jan 9, 2018

I ask you to pay attention to landuse=railway http://www.openstreetmap.org/way/430173017#map=14/51.6062/45.9457&layers=D
its color does not change on 12 zoom

@matkoniecz
Copy link
Contributor

Your intended link is probably http://www.openstreetmap.org/way/430173017#map=14/51.6062/45.9457

@drmx
Copy link

drmx commented Jan 9, 2018

yes. I confused. thank you

@Tomasz-W
Copy link

Tomasz-W commented Nov 24, 2018

@kocio-pl Isn't it resolved by #3466 and other PRs?

@kocio-pl
Copy link
Collaborator

Are you sure all such problems have been fixed?

@Tomasz-W
Copy link

Tomasz-W commented Nov 24, 2018

As amenity areas may be so big that they are visible on zoom levels lower than z13, and allotments should be treated as natural areas, I'm not sure if rest from the objects mentioned above are worth changes because they are usually too small to see them on <z13.

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