Skip to content
This repository has been archived by the owner on Aug 14, 2024. It is now read-only.

Display bridges and piers #122

Merged
merged 2 commits into from
May 8, 2022
Merged

Conversation

Helium314
Copy link
Contributor

fixes #86 and #101

Bridge structures and piers are now displayed with the same color as buildings.

Order is higher than water/landuse and lower than anything else. This means bridges are displayed below ways that actually go under the bridge. Not really nice, but same as in standard style on openstreetmap.org.

@westnordost
Copy link
Member

Or maybe use earth_color?

@Helium314
Copy link
Contributor Author

Maybe... I didn't play around too much with the color. I'll check how it looks.

@Helium314
Copy link
Contributor Author

building_color:
building
earth_color
earth
building_outline_color
building_outline

@Helium314
Copy link
Contributor Author

I also had a look at dark style, and personally I like building_outline_color most in both themes.

@westnordost
Copy link
Member

I think I slightly favour the reddish landuse-color but if you favour the building outline color, that would also be fine.

@Helium314
Copy link
Contributor Author

I'd prefer some color that is not on used for background, because it may look confusing if bridge looks like the area below.
E.g. with earth_color it could be confused with some sort of dam, while a building / building outline color makes it more look like a building.

@westnordost westnordost merged commit 7aab435 into streetcomplete:jawg May 8, 2022
@Joxit
Copy link

Joxit commented May 8, 2022

I'm glad that you added bridges 😁

FYI this is how we render bridges https://maps.jawg.io/#17.94/43.492481/-1.474054

@Helium314 Helium314 deleted the patch-1 branch May 8, 2022 16:09
@mnalis
Copy link
Member

mnalis commented Sep 1, 2022

Was this supposed to render man_made=pier too? Because in SC 45.2 (f-droid) it still doesn't.
For example here is man_made=pier made years ago: https://www.openstreetmap.org/note/3197913#map=19/43.73060/15.80269&layers=N

And it is still not shown in SC 45.2:
small_Screenshot_20220902_003308_de westnordost streetcomplete

@Helium314
Copy link
Contributor Author

It should render, and does work for me

@mnalis
Copy link
Member

mnalis commented Sep 2, 2022

It should render, and does work for me

Huh, did you try at that specific location or somewhere else? Maybe it has some other tags too at place where you tried?

I don't see this one (https://www.openstreetmap.org/way/360995691) on either of my phones (android 10 and android 6), neither on SC 45.2, 45.2EE, nor 46.0 😢 - as picture above shows. Same issue with dark and light themes.

I don't see other piers either (even when I uninstalled and reinstalled SC, so it shouldn't be stale cache issue).
E.g. https://www.openstreetmap.org/way/293594752, https://www.openstreetmap.org/way/598082067 etc.

@Helium314
Copy link
Contributor Author

Helium314 commented Sep 2, 2022

They are not closed ways, and this PR only contains draw for polygons.
You can try adding same thing for lines and see whether it works.

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

Successfully merging this pull request may close these issues.

Piers are not rendered
4 participants