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

Render barriers above highway areas #528

Closed
matthijsmelissen opened this issue May 17, 2014 · 17 comments
Closed

Render barriers above highway areas #528

matthijsmelissen opened this issue May 17, 2014 · 17 comments
Labels
landcover roads wontfix-unfeasible Issues closed because of lack of suitable solution

Comments

@matthijsmelissen
Copy link
Collaborator

Barriers should be rendered above highway areas.

If not, barriers between highway areas become invisible.

Examples:
http://www.openstreetmap.org/#map=18/52.53296/13.37229

See also https://trac.openstreetmap.org/ticket/2807.

@matthijsmelissen
Copy link
Collaborator Author

I think we should be able to change the rendering order, and draw barriers on top of areas.

Should barriers also be drawn on top of buildings, and on top of regular roads?

@Theodin
Copy link

Theodin commented Jun 29, 2014

I think they should. Like benches and other things I cant think of at the moment :)

@matkoniecz
Copy link
Contributor

Drawing on top of regular roads would result in weird things, as rendered road may take more space that in reality. See this hedge for an example: http://www.openstreetmap.org/?mlat=50.07075&mlon=19.89828#map=18/50.07075/19.89828 (and zoom out).

@matthijsmelissen
Copy link
Collaborator Author

Let's decide on #623 first.

@Theodin
Copy link

Theodin commented Jun 30, 2014

@mkoniecz: Isnt that "just" a rendering issue as it depends on how you adjust the size with the zoom levels no matter what is rendered first?

@matkoniecz
Copy link
Contributor

Isnt that "just" a rendering issue as it depends on how you adjust the size with the zoom levels no matter what is rendered first?

Yes, but rendering barrier on top of parallel road is uglier and more confusing than hiding barrier under road.

And at this moment barriers are rendered on top of highway areas (original example is gone, but I found http://www.openstreetmap.org/way/200980711#map=19/50.06105/19.93630).

@matthijsmelissen
Copy link
Collaborator Author

And at this moment barriers are rendered on top of highway areas

No, only in this case because the barrier is explicitly defined as inner element of the multipolygon.

@pmailkeey
Copy link

This looks like a case of the common breach of common sense that states a rendering order of :
largest areas first to smallest areas;
All lines on top of areas
shorter lines on top of longer lines
points over everything.

As otherwise data will be hidden.

The question is, when will this be implemented ?

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 2, 2016

What is current status of this issue and how difficult it would be to make it work?

@matthijsmelissen
Copy link
Collaborator Author

I didn't look into it, simply moving layers around might work.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 2, 2016

We have 79 layers, so it's not trivial to know where to start, even though they have clear names. I believe important ones for this issue are (in order of appearance):

  - id: "line-barriers"
  - id: "area-barriers"
  - id: "highway-area-fill"
  - id: "roads-fill"
  - id: "turning-circle-fill"

My questions:

  1. What is the meaning of layer definitions order - which ones are rendered above?
  2. Is it important to have "-" before "id:" in YAML? Some lines miss it.

Wild guess: probably just moving -barriers (probably without touching water-barriers-) after -fill could be the solution?

Hint - current list of all osm-carto layers:

  - id: "world"
  - id: "coast-poly"
  - id: "builtup"
  - id: "necountries"
  - id: "landcover-low-zoom"
  - id: "landcover"
  - id: "landcover-line"
  - id: "water-lines-casing"
  - id: "water-lines-low-zoom"
  - id: "icesheet-poly"
  - id: "water-areas"
  - id: "landcover-area-symbols"
  - id: "icesheet-outlines"
  - id: "water-lines"
  - id: "water-barriers-line"
  - id: "water-barriers-poly"
  - id: "marinas-area"
  - id: "piers-poly"
  - id: "piers-line"
  - id: "water-barriers-point"
  - id: "bridge"
  - id: "buildings"
  - id: "buildings-major"
  - id: "tunnels"
  - id: "landuse-overlay"
  - id: "line-barriers"
  - id: "cliffs"
  - id: "area-barriers"
  - id: "ferry-routes"
  - id: "turning-circle-casing"
  - id: "highway-area-casing"
    id: "roads-casing"
  - id: "highway-area-fill"
  - id: "roads-fill"
  - id: "turning-circle-fill"
  - id: "aerialways"
  - id: "roads-low-zoom"
  - id: "waterway-bridges"
  - id: "bridges"
  - id: "guideways"
    id: "admin-low-zoom"
  - id: "admin-mid-zoom"
  - id: "admin-high-zoom"
  - id: "power-minorline"
  - id: "power-line"
  - id: "nature-reserve-boundaries"
  - id: "tourism-boundary"
  - id: "trees"
  - id: "country-names"
  - id: "capital-names"
  - id: "state-names"
  - id: "placenames-medium"
  - id: "placenames-small"
  - id: "stations"
  - id: "stations-poly"
  - id: "amenity-points-poly"
  - id: "amenity-points"
  - id: "power-towers"
  - id: "power-poles"
  - id: "roads-text-ref-low-zoom"
  - id: "junctions"
  - id: "bridge-text"
  - id: "roads-text-ref"
  - id: "roads-area-text-name"
  - id: "roads-text-name"
  - id: "paths-text-name"
  - id: "text-poly-low-zoom"
  - id: "text-poly"
  - id: "text-line"
  - id: "text-point"
  - id: "building-text"
  - id: "interpolation"
  - id: "addresses"
  - id: "water-lines-text"
  - id: "ferry-routes-text"
  - id: "admin-text"
  - id: "nature-reserve-text"
  - id: "amenity-low-priority"
  - id: "amenity-low-priority-poly"

@matthijsmelissen
Copy link
Collaborator Author

  1. What is the meaning of layer definitions order - which ones are rendered above?

The order in which they are defined is the order in which they are rendered.

@gravitystorm
Copy link
Owner

  1. Is it important to have "-" before "id:" in YAML? Some lines miss it.

The - indicates the start of the next item in an array. So sometimes the - appears before the id attribute, sometimes before a different attribute (e.g. name):

  - name: "roads-casing"
    id: "roads-casing"
    class: "roads-casing"

@kocio-pl
Copy link
Collaborator

The closer inspection in #2394 shows that it can have nasty side effects on roads, so closing it as wontfix until some sane solution is proposed.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 8, 2019

Reopened; while PR #2394 was not a good solution, it may be possible to address this by rendering highway areas earlier, as in #3872

@jeisenbe jeisenbe reopened this Sep 8, 2019
@jeisenbe
Copy link
Collaborator

jeisenbe commented Nov 2, 2019

#3872 was closed, so it appears this issue will not be fixed

@jeisenbe jeisenbe closed this as completed Nov 8, 2019
@BertMule
Copy link

BertMule commented Nov 6, 2022

I also miss this feature along the sense of pmailkeey.

Examples:

  • Hedge on a pedestrian area, like I would expect just like a tree row or trees. All visible in Amersfoort currently.
  • Walls on a pedestrian area. As in Makkum.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
landcover roads wontfix-unfeasible Issues closed because of lack of suitable solution
Projects
None yet
Development

No branches or pull requests

8 participants