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

amenity=university label should be more prominently displayed #287

Closed
matkoniecz opened this issue Dec 13, 2013 · 7 comments
Closed

amenity=university label should be more prominently displayed #287

matkoniecz opened this issue Dec 13, 2013 · 7 comments

Comments

@matkoniecz
Copy link
Contributor

way link where I encountered this problem: http://www.openstreetmap.org/way/157902552

problem: to replicate go to http://www.openstreetmap.org/?mlat=50.06578&mlon=19.91920#map=19/50.06578/19.91920 and zoom out

label is visible on zoom levels 19, 18, 17, 16, 14

should be also visible

on 15 zoom level but apparently house name managed to block appearance of label text, amenity=university label should have higher priority

on 13 zoom level - probably using short_name

@pnorman
Copy link
Collaborator

pnorman commented Dec 14, 2013

z14 is coming from the catch-all. I suspect the same is happening on z15, and the catch-all is the lowest priority.

@matkoniecz
Copy link
Contributor Author

now it is also visible on zoom level 15, still invisible on level 13

@dieterdreist
Copy link

I have encountered a similar problem: the university name is only rendered in zoom 19. It seems maybe the parking symbol (access=private) is preventing that the name is rendered. My suggestion is to render the parking symbol later (or maybe allow overlapping of labels for the combination with access=private):
http://www.openstreetmap.org/#map=19/41.86037/12.49663
http://www.openstreetmap.org/relation/1517075

@ukasiu
Copy link

ukasiu commented Jan 29, 2014

Similar problem as dieterdreist's here http://www.openstreetmap.org/way/257147754

@Klumbumbus
Copy link

Here is one more example where parking blocks the rendering of the university area label: http://www.openstreetmap.org/?mlat=50.81602&mlon=12.93069#map=19/50.81602/12.93069
The name of the university area http://www.openstreetmap.org/way/24518628 is rendered only at zoom 19.
The parking area is rendered on top of the university area, which is fine. However the label of amenity=university should have more priority in rendering then the icon of amenity=parking.

@dieterdreist
Copy link

here as well, and the parking is even a private one:
http://www.openstreetmap.org/?mlat=50.81602&mlon=12.93069#map=18/41.86018/12.49709

matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Aug 6, 2014
POI are now ordered by way area. Note that icons are never rendered on top
of each other, so this has only influence on priority decision of icon
rendering.

Large POI (such as universities, malls, etc.) are now rendered with higher
priority.

The old order was:
* Render points with higher priority than areas
* Icon priority from high to low: tourism, amenity, highway, waterway,
  man_made, historic, shop, leisure
* Text priority from high to low: place, amenity, natural, historic, leisure,
  landuse, tourism, waterway, man_made, shop, aeroway
It is clear that the old order was quite a mess.

* Resolves partially gravitystorm#287
* Resolves partially gravitystorm#797
* Resolves gravitystorm#802
* Resolves https://trac.openstreetmap.org/ticket/4100
matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Aug 7, 2014
POI are now ordered by way area. Note that icons are never rendered on top
of each other, so this has only influence on priority decision of icon
rendering.

Large POI (such as universities, malls, etc.) are now rendered with higher
priority.

The old order was:
* Render points with higher priority than areas
* Icon priority from high to low: tourism, amenity, highway, waterway,
  man_made, historic, shop, leisure
* Text priority from high to low: place, amenity, natural, historic, leisure,
  landuse, tourism, waterway, man_made, shop, aeroway
It is clear that the old order was quite a mess.

* Resolves partially gravitystorm#287
* Resolves partially gravitystorm#797
* Resolves gravitystorm#802
* Resolves https://trac.openstreetmap.org/ticket/4100
matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Aug 27, 2014
POI are now ordered by way area. Note that icons are never rendered on top
of each other, so this has only influence on priority decision of icon
rendering.

Large POI (such as universities, malls, etc.) are now rendered with higher
priority.

The old order was:
* Render points with higher priority than areas
* Icon priority from high to low: tourism, amenity, highway, waterway,
  man_made, historic, shop, leisure
* Text priority from high to low: place, amenity, natural, historic, leisure,
  landuse, tourism, waterway, man_made, shop, aeroway
It is clear that the old order was quite a mess.

* Resolves partially gravitystorm#287
* Resolves partially gravitystorm#797
* Resolves gravitystorm#802
* Resolves https://trac.openstreetmap.org/ticket/4100
matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Aug 28, 2014
POI are now ordered by way area. Note that icons are never rendered on top
of each other, so this has only influence on priority decision of icon
rendering.

Large POI (such as universities, malls, etc.) are now rendered with higher
priority.

The old order was:
* Render points with higher priority than areas
* Icon priority from high to low: tourism, amenity, highway, waterway,
  man_made, historic, shop, leisure
* Text priority from high to low: place, amenity, natural, historic, leisure,
  landuse, tourism, waterway, man_made, shop, aeroway
It is clear that the old order was quite a mess.

* Resolves partially gravitystorm#287
* Resolves partially gravitystorm#797
* Resolves gravitystorm#802
* Resolves https://trac.openstreetmap.org/ticket/4100
@matkoniecz
Copy link
Contributor Author

Original problem was fixed by #860 - there is still issue with unimportant icon blocking important label. I will create a new ticket for this.

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

6 participants