Modify

Opened 2 years ago

Last modified 2 years ago

#23463 new defect

unnecessary.mapcss suggests removing area=yes from railway=platform for indoor features

Reported by: rskedgell Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: Cc:

Description

While area=yes may be unnecessary for most indoor objects, the wiki explicitly requires it for railway=platform - https://wiki.openstreetmap.org/wiki/Tag:railway%3Dplatform

Changing line #24 of unnecessary.mapcss from
way:closed[indoor ][area?][!highway][indoor!=no]
to
way:closed[indoor ][area?][!highway][indoor!=no][railway!=platform]
would fix this, but there are probably other cases where area=yes should be preserved.

Attachments (0)

Change History (3)

comment:1 by anonymous, 2 years ago

Component: CoreCore validator

comment:2 by taylor.smock, 2 years ago

From the original proposal:

Platforms can also be mapped as areas, for better rendering results a closed way should also be tagged as
area=yes
for platforms used in bus stations or by non rail services

I'm strongly disinclined to special case a tag combination that was specifically created for rendering purposes; a closed polygon should be more than enough of a hint that the platform is drawn as an area.

comment:3 by skyper, 2 years ago

Well indoor is kind of special as it is the lonesome line which does not include a primary key. While I think area=* is not needed for railway=platform I wonder why we flag railway=platform only in combination with indoor.

There are not many tags which can be both, an area or a linear object, mapped as a closed way but leisure=track is one of them and at least [leisure!=track] is missing.

Last edited 2 years ago by skyper (previous) (diff)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to rskedgell.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.