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

man_made=bridge layer issue #1799

Closed
simonpoole opened this issue Aug 30, 2015 · 11 comments
Closed

man_made=bridge layer issue #1799

simonpoole opened this issue Aug 30, 2015 · 11 comments

Comments

@simonpoole
Copy link

First thanks a lot for supporting the tag in the first place.

There seems to be a minor problem with respect to supporting the layer tag correctly wit http://www.openstreetmap.org/#map=18/47.38605/8.53404

@matkoniecz
Copy link
Contributor

For now it is like buildings - displayed below all roads.

Partial reason is to show roads, junctions etc below bridges (like at http://www.openstreetmap.org/?mlat=50.04999&mlon=19.93513#map=19/50.04999/19.93513) - but I am not sure is it a good reason to keep it this way, it is probably really rare to see situations where there is something interesting below and it is readable.

177696

Second reason that it was relatively easy to code displaying it like building. To handle layers it would be much more complicated - it would be part of SQL query starting at

(SELECT way, (CASE WHEN substr(feature, length(feature)-3, 4) = 'link' THEN substr(feature, 0, length(feature)-4) ELSE feature END) AS feature,
and ending 119 lines later.

@matkoniecz matkoniecz added this to the Bugs and improvements milestone Aug 30, 2015
@matkoniecz
Copy link
Contributor

@simonpoole In provided example there is a better situation - this steps may be hidden on implementig this suggestion (and somebody would probably change layers to tag it for renderer - and encouraging things like this would be a poor idea)
91789

@matkoniecz
Copy link
Contributor

show roads, junctions etc below bridges

I think that it is a good enough reason to layer it like buildings. I plan on closing this ticket as IMHO current solution is not perfect - but there is no clear way to improve this.

@dieterdreist
Copy link

sent from a phone

Am 30.08.2015 um 22:22 schrieb Mateusz Konieczny notifications@github.com:

In provided example there is a better situation - this steps may be hidden on implementig this suggestion (and somebody would probably change layers to tag it for renderer - and encouraging things like this would be a poor idea)

in this example the bridge appears to be below the crossing road, thereby confusing the map reader. I agree with Simon that there is still a flaw in bridge rendering. Maybe there could be an exception for using transparency in conjunction with bridges?

@Gazer75
Copy link

Gazer75 commented Sep 3, 2015

Tbh the entire layering of buildings vs roads can be confusing, but I guess as this render is focused on roads it can't be helped.
Would suggest not even displaying the bridge building at all. Just makes it all more confusing when roads are layered on top.

@gravitystorm
Copy link
Owner

Maybe there could be an exception for using transparency

I've said this before and I'll say it again. No transparency. :-)

#552 (comment)

You don't actually want to blend the road colour and the bridge colour together to get a darker murky brown. What you are actually wanting to do is find the roads that (physically) lie below the bridge, and colour those bits with a different (perhaps lighter? tunnel-like?) colour, while still drawing the bridge polygon first and the roads after. So think st_intersects() and st_split() and things like that, not translucency.

I'm going to close this as per @matkoniecz since it's not a huge problem and the fix would be quite challenging. Of course, if a PR appears with an alternative approach, that would be of interest.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Sep 4, 2015

@gravitystorm I'm confused then what is translucency/transparency, what is not, and why it is considered a bad thing. Using examples:

  1. "French style" tree canopy uses it or not (opacity=0.3 IIRC)?
  2. MapQuest Open uses it or not? Eiffel Tower, Berlin Hauptbahnhof or even Palace of Culture and Science in Warsaw all look nice and even better than on osm-carto - or do you think they are flawed maybe?

Closed bridge-related issue may be not the best place to ask such general questions, but we don't have special place for it, so sorry for "hijacking" it...

@gravitystorm
Copy link
Owner

opacity/transparency/translucency are all similar things in our conversations (https://en.wikipedia.org/wiki/Transparency_and_translucency has the details.

Yes, the tree canopies use opacity. MapQuest Open uses transparency almost everywhere and that leads to terrible colour mixing - it's a really bad idea, and we knew that when I worked on that style a few years ago. (For your examples, ask why are some buildings darker than others?) But for the best explanation of why it is bad, see my link above regarding footpaths and roofs.

@dieterdreist
Copy link

Here's another example that demonstrates that we should take the layer value for bridge objects into account for the rendering order:

layer_ponte_marconi

http://www.openstreetmap.org/way/44097288#map=19/41.86046/12.47039

@pnorman
Copy link
Collaborator

pnorman commented Nov 28, 2015

Here's another example that demonstrates that we should take the layer value for bridge objects into account for the rendering order:

Could you do a PR with a fix, so we can have a look at how much complexity it adds?

@sommerluk
Copy link
Collaborator

Duplicate of #2509 and #3203

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

No branches or pull requests

8 participants