Opened 12 years ago
Closed 11 years ago
#7722 closed defect (fixed)
do not render junction=roundabout as area
Reported by: | skyper | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Internal mappaint style | Version: | latest |
Keywords: | junction roundabout | Cc: |
Description
Attachments (1)
Change History (15)
comment:1 Changed 12 years ago by
comment:3 Changed 12 years ago by
This line is outdated:
trunk/styles/standard/elemstyles.xml?rev=5247#L742
Note: junction=roundabout can be used on nodes but please never use mini_roundabouts for normal ones as these are two different things. See: http://lists.openstreetmap.org/pipermail/tagging/2012-May/010107.html
Changed 12 years ago by
Attachment: | 7722.patch added |
---|
comment:4 follow-up: 5 Changed 12 years ago by
Summary: | do not render junction=roundabout as area → [Patch] do not render junction=roundabout as area |
---|
I tried to stay away from the roundabout discussion … ;-)
skyper, I'd …
- allow
junction=roundabout
on nodes + icon - require
highway
only forjunction
s on ways (validator) - remove area style for
highway=turning_circle
… do you agree?
comment:5 follow-up: 6 Changed 12 years ago by
Replying to simon04:
I tried to stay away from the roundabout discussion … ;-)
skyper, I'd …
- allow
junction=roundabout
on nodes + icon- require
highway
only forjunction
s on ways (validator)- remove area style for
highway=turning_circle
… do you agree?
Thanks, that is fine, but please delete the second part of source:trunk/styles/standard/elemstyles.xml?rev=5247#L742.
comment:6 follow-up: 8 Changed 12 years ago by
Replying to skyper:
Thanks, that is fine, but please delete the second part of source:trunk/styles/standard/elemstyles.xml?rev=5247#L742.
Already covered in the patch. :-)
comment:8 Changed 12 years ago by
Replying to simon04:
Replying to skyper:
Thanks, that is fine, but please delete the second part of source:trunk/styles/standard/elemstyles.xml?rev=5247#L742.
Already covered in the patch. :-)
Sorry, should have look at it. :-)
comment:9 Changed 12 years ago by
Summary: | [Patch] do not render junction=roundabout as area → do not render junction=roundabout as area |
---|
comment:11 follow-up: 12 Changed 11 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Now all roundabouts are rendered white instead of the colour according to highway=*
We do not have to evaluate junction=roundabout for rendered colour but simple stick to the highway rendering.
EDIT: It is also not possible to use the preset for several ways forming a circle.
comment:12 Changed 11 years ago by
Replying to skyper:
Now all roundabouts are rendered white instead of the colour according to highway=*
We do not have to evaluate junction=roundabout for rendered colour but simple stick to the highway rendering.
So basically, we only render roundabouts on nodes? The oneway arrows are determined differently.
Happens with a closed way.
A closed way with highway=* should never be treated as area unless it is tagged with area=yes.