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

Revised text attributes and size of natural=ridge and natural=arete #470

Merged
merged 1 commit into from
Jan 22, 2018

Conversation

Ircama
Copy link
Contributor

@Ircama Ircama commented Dec 5, 2017

Revised text attributes and size of natural=ridge and natural=arete

Now polylines are rendered only from zoom 6 to zoom 10; in order to show them at these zooms, the maps provided with Osmand shall be complemented with a specific layer (as the one attached here).

Removed rendering of polylines from zoom 11 on (as done with valleys).

@Ircama Ircama force-pushed the ircama-ridge-no-text branch from 7105a0a to 44f6149 Compare December 10, 2017 10:56
Ircama added a commit to Ircama/OsmAnd-resources that referenced this pull request Dec 10, 2017
Using a similar pattern to [natural=ridge and natural=arete](osmandapp#470).
Ircama added a commit to Ircama/OsmAnd-resources that referenced this pull request Dec 10, 2017
Using a similar pattern to [natural=ridge and natural=arete](osmandapp#470).
@Ircama Ircama force-pushed the ircama-ridge-no-text branch from 44f6149 to 56cbad4 Compare December 10, 2017 11:56
@xmd5a2
Copy link
Contributor

xmd5a2 commented Dec 12, 2017

Don't agree with removing rendering of ridges. Contour lines are often not enough to represent them because of accuracy.

xmd5a2 pushed a commit that referenced this pull request Dec 12, 2017
Using a similar pattern to [natural=ridge and natural=arete](#470).
@Ircama
Copy link
Contributor Author

Ircama commented Dec 12, 2017 via email

@Ircama Ircama force-pushed the ircama-ridge-no-text branch from 56cbad4 to 063f747 Compare January 21, 2018 11:10
@Ircama
Copy link
Contributor Author

Ircama commented Jan 21, 2018

Sorry for this delay, unfortunately depending on my limited free time.
I revised the code by enabling a thin line at all zooms.
Please check samples below.

screenshot_20180121-115720

screenshot_20180121-115729

screenshot_20180121-115751

screenshot_20180121-115808

screenshot_20180121-115827

screenshot_20180121-115848

screenshot_20180121-115907

screenshot_20180121-115921

screenshot_20180121-115935

screenshot_20180121-115949

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.

2 participants