Opened 11 years ago
Closed 11 years ago
#9144 closed defect (fixed)
Always treat multipolygons as area
Reported by: | skyper | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 14.05 |
Component: | Internal mappaint style | Version: | latest |
Keywords: | multipolygon area | Cc: |
Description
Multipolygons should be always treated as areas despite being tagged with area=*
.
Just try public_transport=platform
or any highway=*
without area=yes
.
Attachments (0)
Change History (12)
follow-up: 4 comment:1 by , 11 years ago
comment:2 by , 11 years ago
I suggest to remove the general style for area=yes
. The style should come from additional tags that have a concrete meaning.
follow-up: 8 comment:3 by , 11 years ago
But area=yes/no can modify the specific tags' meaning. So it should be rendered visibly that highway=pedestrian+area=yes is different from highway=pedestrian. Or leisure=track+area=no different from leisure=track.
Also, I've created some polygons just having area=yes+surface=*, those getting the nice filled in style. What is the right way to tag those, when it is not some specific object, just concrete or sandy surface on the ground?
follow-up: 7 comment:4 by , 11 years ago
Replying to bastiK:
In 7154/josm:
Why did you change the rendering for public_transport=platform
closed ways ? They are always an area.
Replying to bastiK:
I suggest to remove the general style for
area=yes
. The style should come from additional tags that have a concrete meaning.
+1
In 95% the geometry is enough and a polygon is an area. area=yes
might be needed only in rare cases and even for highway=*
area:highway=*
was proposed. I would say area=no
would be the appropriate tag to mark a polygon which is no area.
Replying to aceman:
But area=yes/no can modify the specific tags' meaning. So it should be rendered visibly that highway=pedestrian+area=yes is different from highway=pedestrian. Or leisure=track+area=no different from leisure=track.
Also, I've created some polygons just having area=yes+surface=*, those getting the nice filled in style. What is the right way to tag those, when it is not some specific object, just concrete or sandy surface on the ground?
How about landcover=*
with out any area=*
.
comment:7 by , 11 years ago
Replying to skyper:
Why did you change the rendering for
public_transport=platform
closed ways ? They are always an area.
I've changed it back.
Replying to bastiK:
I suggest to remove the general style for
area=yes
. The style should come from additional tags that have a concrete meaning.
+1
In 95% the geometry is enough and a polygon is an area.
area=yes
might be needed only in rare cases and even forhighway=*
area:highway=*
was proposed. I would sayarea=no
would be the appropriate tag to mark a polygon which is no area.
Done. We should now handle each tag separately if area=yes/no
impacts the styling.
follow-up: 9 comment:8 by , 11 years ago
Replying to aceman:
But area=yes/no can modify the specific tags' meaning. So it should be rendered visibly that highway=pedestrian+area=yes is different from highway=pedestrian. Or leisure=track+area=no different from leisure=track.
Should work now, please note all tags that require special handling.
Also, I've created some polygons just having area=yes+surface=*, those getting the nice filled in style. What is the right way to tag those, when it is not some specific object, just concrete or sandy surface on the ground?
Perhaps add a style that visualizes surface? There are several in the repository. :)
comment:9 by , 11 years ago
Replying to bastiK:
Replying to aceman:
But area=yes/no can modify the specific tags' meaning. So it should be rendered visibly that highway=pedestrian+area=yes is different from highway=pedestrian. Or leisure=track+area=no different from leisure=track.
Should work now, please note all tags that require special handling.
I am not sure why there would be only some tags where area=yes/no would be meaningful. On the wiki there is no distinction, only the defaults are listed. But those defaults can be overriden in either way. So even landuse=*+area=no should be possible even though that really is probably not very useful. But maybe somebody find a use :) So the handling should be universal and allow all combinations as the theory says.
comment:10 by , 11 years ago
Milestone: | → 14.05 |
---|
comment:12 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
area=no
should now work universally.
For area=yes
, the question is whether a generic light gray is really that helpful.
In 7154/josm: