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

Rendering of strait from z10 #3438

Merged
merged 1 commit into from
Oct 11, 2018
Merged

Rendering of strait from z10 #3438

merged 1 commit into from
Oct 11, 2018

Conversation

jragusa
Copy link
Contributor

@jragusa jragusa commented Oct 7, 2018

Fixes #804

Changes proposed in this pull request:
Add label to natural=strait from z10

Test rendering with links to the example places:
https://www.openstreetmap.org/relation/2667319

Before
strait_before

After
strait_after

@jragusa
Copy link
Contributor Author

jragusa commented Oct 7, 2018

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 7, 2018

Great, thanks for picking it up!

Please make a test on some other node, because this one renders place=locality label (it should be probably removed as a tagging for rendering):

https://www.openstreetmap.org/node/619215170

@jragusa
Copy link
Contributor Author

jragusa commented Oct 7, 2018

Oups, I missed that. Corrected rendering from z17:
strait_point_z17_corrected

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 7, 2018

@jeisenbe
Copy link
Collaborator

jeisenbe commented Oct 7, 2018 via email

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 8, 2018

I'm not aware of any linear rendering for such things, only nodes and areas (closed ways or relations) - this code is shared for bays for example.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Oct 8, 2018 via email

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 8, 2018

I think adding linear rendering is a task for a separate PR (it will be good to check if some other similar objects might be rendered like that).

@kocio-pl kocio-pl merged commit cc3c271 into gravitystorm:master Oct 11, 2018
@kocio-pl
Copy link
Collaborator

Thanks a lot for this code! Works as expected.

Now if anybody would like to research how to support linear shapes, it would be next big step...

@geozeisig
Copy link

I found examples of areas (example) but none for ways (example)? Maybe it will take time. Would it not be better to do the writing in the direction of the way? As it is already done at the ferry lines.

@jragusa
Copy link
Contributor Author

jragusa commented Oct 21, 2018

@geozeisig, look at the cap horn for ways
strait_overpass

@geozeisig
Copy link

It looks like natural=strait is not rendered on ways.

@HolgerJeromin
Copy link
Contributor

This was mentioned before:

I think adding linear rendering is a task for a separate PR (it will be good to check if some other similar objects might be rendered like that).

@geozeisig
Copy link

Rendering on linear ways could be done the same as with rivers. This could determine the direction of the font. It would be repeated at long ways. Only the font size could be larger in many places, which could be a problem to assess correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants