-
Notifications
You must be signed in to change notification settings - Fork 822
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
Filling man_made=pier with color #3459
Comments
This was discussed in #2652, but I still don't find land-colour fill a proper one for this feature. |
It's also discussed in #3343 where I proposed to use the parking colour. |
Hm, I see nothing wrong with a current color.
What difference do you see? I'm not aware of anything like that, we just made linear piers thinner in #3267, but as far as I know nothing has changed for areas, especially the color. |
In my case it's like when I see land colour, I'm intuively treating it as "there is nothing here". Of course it's good to have at least land colour to see piers on the water, but I think we shouldn't use land colour for any other feature than a blank land. |
I like the idea of some kind of brown color. 4049 piers have the surface tag, of which 2981 are tagged as wood. There's 318,281 of them overall though. So it might not reflect the surface of piers in general, but I do think the default surface for most people when they think of piers is wood, whatever material any particular pier might be made out of in real life. |
My idea would be to start with something subtle, just to show there's not bare land, like different small x (like 1, 2, 5, 10...) in:
|
When I'm doing my amateur test renderings in Photoshop, I'm using usually "round" numbers by moving each value by -5 or +5, so I suggest with trying 5, 10, 15% etc., otherwise it would be a straight way to go insane, because as we are mostly men here, we would propably have big problems with rating such subtle colour differences (current looking for minor/ major buildings proper shades is an exeption) ;) |
I would not drive colour decisions from the surface material here. Probably my parking comparison wasn't a good argument either. But a pier is not land, it is a built feature. (current @land-color is f2efe9, thus not neutral grey) |
I just don't want another Beigeish/off yellowish color. And it would be cool if it was clear where the land stops and the pier starts. Its not so clear with fff3d7. At least in the example. What about a variation of the footway area or leisure colors? Either that, or we can go with something slightly lighter then parking like @polarbearing suggested. |
Thanks. I was imagining something a little lighter, but its to bad. People would definitely know they were there. Although it might scrue up rendering of amenity stuff due to also being brown. |
I like the last one. Id be willing to test it when I have the time if know one beats me to it. Thanks for doing the mockups. |
I don't think any of the above renderings are an improvement over what we have now. Perhaps instead of changing the fill color the casing color could be tweaked? (like a bridge casing) I would like to leave the pier color alone and allow combining it with |
I find the last render shown by Tomasz-W called bridge fill re-use an improvement. It shows there is something rather than nothing. |
What would that mean exactly? |
Basing on a visualisations above I think that only '3% darken' and 'the same fill with some outline' versions are worth further testing. |
I'm afraid that piers are so thin, that the outline would dominate them too much, especially at lower zoom levels. Bridges are typically wider. But maybe it would be enough to move the piers layers (like |
I think that's just the point. Piers are nothing. They are just an extension of land that goes out a little farther over the water. If there is "something" on them, that should be rendered separately. While bridge casing may work where the water meets the pier, I do not think it would look good to have a casing where the pier meets the land (for piers mapped as an area). |
That would require rendering piers below landuse colors. However, it would also mean that the parts of piers on top of beaches would become yellow (de facto invisible). |
They are not nothing, they are man_made objects. The can be party over land and are partly over water. |
I've mentioned a possible way to render landcover over piers in #330 (comment) - this requires implementing #3854 first. |
man_made=pier does not get rendered anymore, it's looks like rendering blancs:
https://www.openstreetmap.org/way/451067705
Is that a conscious choice or did something go wrong?
The text was updated successfully, but these errors were encountered: