[landcover] surface=sand #2873
Comments
Author: JohnSmith After further comments on the tagging list it should be surface=sand. |
Author: dieterdreist Replying to [comment:1 JohnSmith]:
After further comments on the tagging list this should be landcover=sand. |
Author: JohnSmith Replying to [comment:2 dieterdreist]:
Why should it, so far no one has shown how landcover=* is better, in fact it just makes things worst because it ends up duplicating surface=* for no good reason and surface=* is widely used already. |
Author: Ldp Replying to [comment:2 dieterdreist]:
After further comments off list we're now rendering natural=sand. |
Author: JohnSmith Replying to [comment:4 Ldp]:
And this is why this bug was filed, natural=sand might be ok for a beach, however it doesn't fit for things like beach volleyball courts and golf bunkers. |
Author: Ldp You're right. For those you would use, in order:
In short, surface is regarded as a supporting tag, giving extra information. It is not a main tag in and of itself. We can't render surface=sand as-is, without there being a main tag to say what feature has a sand surface. And even then, we're likely to just render that main tag, not taking any surface=* into account. |
Author: JohnSmith Replying to [comment:6 Ldp]:
Why does it matter what the main tag is, any area with surface=sand just needs to be a yellowish colour then you don't need to know/care what the main tag is, and it will still render. |
Author: dieterdreist IMHO surface=sand will not work on natural=beach, because there will also be buildings and paved areas on beaches, which are not surface sand, but still they are on the beach (so natural=beach will include the buildings and paved areas, surface=sand won't). |
Author: JohnSmith Replying to [comment:8 dieterdreist]:
That's grasping at straws, since overlapping polygons is already solved. Assuming of course that the polygons overlap. |
Author: math1985 This has been resolved in the meanwhile. |
Reporter: JohnSmith
[Submitted to the original trac issue database at 7.26am, Saturday, 10th April 2010]
Please render natural=sand the same as natural=beach
The text was updated successfully, but these errors were encountered: