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

Render themes: line symbol #124

Closed
andreynovikov opened this issue Aug 12, 2016 · 13 comments
Closed

Render themes: line symbol #124

andreynovikov opened this issue Aug 12, 2016 · 13 comments
Milestone

Comments

@andreynovikov
Copy link

https://github.com/mapsforge/vtm/blob/master/vtm/src/org/oscim/theme/XmlThemeBuilder.java#L258

Useful for aerialway icons.

@devemux86
Copy link
Collaborator

devemux86 commented Aug 12, 2016

And oneways too!

lineSymbol is coming from Mapsforge and I'm not sure we need to have it as a new tag.
Line texture can suffice, provided that we enhance the stipple attributes with a repeat-gap.
Currently stipple attribute plays the role of both the texture length and the gap.

@andreynovikov
Copy link
Author

Not really. Line texture is aligned (rotated) along line. Line symbol should remain vertical, like node symbol.

@devemux86
Copy link
Collaborator

devemux86 commented Aug 12, 2016

Actually in Mapsforge we have an extra attribute rotate for that, with default behavior to be aligned with the line.

@andreynovikov
Copy link
Author

Then ok, will wait for implementation.

@Klaus-Tockloth
Copy link

I use lineSymbol for cliffs and for one-way streets. It would be great if this feature could be implemented.

bildschirmfoto 2016-10-29 um 21 07 17

bildschirmfoto 2016-10-29 um 21 06 31

@devemux86 devemux86 changed the title lineSymbol style not implemented Render themes: line symbol Sep 3, 2017
@devemux86 devemux86 added this to the 0.9.0 milestone Sep 3, 2017
@devemux86
Copy link
Collaborator

Thanks to PR #388 by @Longri for Mapsforge theme compatibility, we have now an implementation of line symbols also in VTM themes.

@devemux86
Copy link
Collaborator

cc @Longri
Oneways don't render correctly, could need more work.

@devemux86
Copy link
Collaborator

I pushed the needed fixes for line symbols.

This was referenced Sep 23, 2017
@devemux86
Copy link
Collaborator

Oneways with the lineSymbol rule:

oneways

@andreynovikov
Copy link
Author

Have tried this new feature for cliffs and failed. Looks like repeat-gap with value less then 30 does not work. I've thought it's because of repeat-start, have set it to 0 but still get too large gaps between symbols.

@devemux86
Copy link
Collaborator

Isn't the natural=cliff rendering supposed to be with symbols having some distance among them?

Also it depends on symbols quality. If the symbols have much empty space around in their canvas, then they cannot be much close, even if put small repeat-gap.

@andreynovikov
Copy link
Author

Yes. But the distance is too large. I use square icon with a tick: https://github.com/andreynovikov/maptrek/blob/master/app/src/main/assets/symbols/natural/cliff.svg There is no much space around. But the gap between symbols is about 1 centimeter on smartphone screen.

@devemux86
Copy link
Collaborator

Trying even with the oneway symbol which doesn't have empty canvas at its width and with repeat-gap="0" repeat-start="0", we cannot achieve continuous symbols rendering.

That seems related to the underlying stipples mechanism and not with line symbols (using it).
You can create a new issue to check it there.

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

No branches or pull requests

3 participants