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

Mall names aren't shown when they have shops #797

Closed
matthijsmelissen opened this issue Aug 1, 2014 · 2 comments · Fixed by #1071
Closed

Mall names aren't shown when they have shops #797

matthijsmelissen opened this issue Aug 1, 2014 · 2 comments · Fixed by #1071
Labels
amenity-points
Milestone
Bugs and improvem...

Comments

@matthijsmelissen
Copy link
Collaborator

When the shops inside the malls are mapped, the mall name is either not shown at all, or shown as small as the shop names. I guess it should take precedence above the little shops.

Also, as malls are commonly pretty big structures and often serve as reference points, their names should be shown from zoom level 15 or 16 (depending on their size, if possible).

See also https://trac.openstreetmap.org/ticket/3440.
@mverwijmeren
Copy link

This issue is just a variation on #759.

matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Aug 6, 2014
@matthijsmelissen
Order POI by way_area
80517e5 
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
@matthijsmelissen
Order POI by way_area
918e827 
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 matthijsmelissen mentioned this issue Aug 7, 2014
Merged
matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Aug 27, 2014
@matthijsmelissen
Order POI by way_area
a64d56d 
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
@matthijsmelissen
Order POI by way_area
fe1d623 
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

As #860 was merged - is it still happening?

@matthijsmelissen matthijsmelissen modified the milestones: Bugs and improvements, 3.x - Needs upgrade to mapnik or openstreetmap-carto.style Sep 26, 2014
matthijsmelissen added a commit to matthijsmelissen/openstreetmap-carto that referenced this issue Oct 18, 2014
@matthijsmelissen
Render shop=mall label like landuse=retail label
4e14439 
* Resolves gravitystorm#759
* Resolves gravitystorm#797
@matthijsmelissen matthijsmelissen mentioned this issue Oct 18, 2014
Merged
@matkoniecz matkoniecz modified the milestones: Bugs and improvements, 3.x - Needs upgrade to openstreetmap-carto.style Mar 24, 2016
@openstreetmap-trac openstreetmap-trac mentioned this issue Jul 23, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
amenity-points
Projects
None yet
Milestone
Bugs and improvements
Development

Successfully merging a pull request may close this issue.

Render shop=mall label like landuse=retail label matthijsmelissen/openstreetmap-carto
3 participants
@matkoniecz @mverwijmeren @matthijsmelissen