Skip to content
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

Semicolons are not correctly handled on aeroway=runway #1035

Closed
sb12 opened this issue Oct 10, 2014 · 7 comments
Closed

Semicolons are not correctly handled on aeroway=runway #1035

sb12 opened this issue Oct 10, 2014 · 7 comments

Comments

@sb12
Copy link
Contributor

sb12 commented Oct 10, 2014

runway

http://www.openstreetmap.org/way/31724226

Probably introduced by #750

@pnorman
Copy link
Collaborator

pnorman commented Oct 10, 2014

Hmm. There's also a data error here, runway refs should with a slash. For this particular bug, I don't see an obvious cause to why it would work on roads but not runways

@sb12
Copy link
Contributor Author

sb12 commented Oct 10, 2014

You are right. This is probably in most of the about 60 cases in the world a data error. Not sure if there's any case where semicolons in runway or taxiway refs are valid.

The problem seems to be the text-placement=line for taxiways and runways.

@matthijsmelissen
Copy link
Collaborator

The problem seems to be the text-placement=line for taxiways and runways.

This probably means that aeroways should be excluded from the multi-line scheme.

@pnorman
Copy link
Collaborator

pnorman commented Oct 10, 2014

This probably means that aeroways should be excluded from the multi-line scheme

Yes - and the rendering probably isn't the most sensible for them either.

Perhaps we should make a new layer for them, and exclude aeroway=* from the shields query for roads? I'm thinking of the uncommon case where a runway is also a road.

@pnorman
Copy link
Collaborator

pnorman commented Oct 10, 2014

The problem seems to be the text-placement=line for taxiways and runways.

We should probably report this upstream, as it looks like a bug even if we don't want to be doing multi-line ref rendering from them.

@RobJN
Copy link

RobJN commented Oct 30, 2014

This bug was also raised in an OSM diary post:
http://www.openstreetmap.org/user/Vid%20the%20Kid/diary/25989

@pnorman
Copy link
Collaborator

pnorman commented Jul 16, 2022

The upstream bug with the unknown character is fixed
image

I'd like to not apply the ; newline logic to aeroways, but splitting that text rendering into its own layer is covered by #2169

@pnorman pnorman closed this as completed Jul 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants