-
Notifications
You must be signed in to change notification settings - Fork 831
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
Add rendering for man-made cranes #3501
Merged
Merged
Changes from 27 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
0f9646b
Merge pull request #1 from gravitystorm/master
jeisenbe 11543b1
Added amenity=internet_cafe
jeisenbe 8a201e3
added amenity=internet_cafe
jeisenbe 5ec0f0c
added icon for amenity=internet_cafe from Tomasz-W
jeisenbe 0e8bb15
Rename amenity=internet_cafe.svg to internet_cafe.svg
jeisenbe 445e091
Merge pull request #2 from jeisenbe/master
jeisenbe 5ab0c1a
Delete internet_cafe.svg
jeisenbe 76d8b64
Add new internet_cafe.svg from Tomasz-W
jeisenbe 57d1cca
Merge pull request #3 from jeisenbe/internet-cafe
jeisenbe f5eceeb
Update amenity-points.mss
jeisenbe e2b6e32
internet_cafe
e481bfe
readded
8f80832
Merge branch 'master' of https://github.com/jeisenbe/openstreetmap-carto
jeisenbe b5e3167
Delete .DS_Store
jeisenbe 1b321a2
Removed .DS_Store files
jeisenbe e90d3ea
Merge branch 'master' of https://github.com/jeisenbe/openstreetmap-carto
jeisenbe 33f9472
Merge branch 'master' of https://github.com/gravitystorm/openstreetma…
jeisenbe 8166b3c
Merge remote-tracking branch 'upstream/master'
jeisenbe 771bd7a
Merge remote-tracking branch 'upstream/master'
jeisenbe 4bc0cf1
Merge branch 'master' of https://github.com/jeisenbe/openstreetmap-carto
jeisenbe c4b8d0c
Add man_made_crane rendering
jeisenbe 8e048fe
Merge remote-tracking branch 'upstream/master' into crane
jeisenbe bbad22e
Add marker-transform for crane icon
jeisenbe c2e0af6
Add marker-transform for crane icon
jeisenbe 8af5fa1
Reduce crane initial zoom to z14 for >100m
jeisenbe 231af2f
Merge branch 'crane' of https://github.com/jeisenbe/openstreetmap-car…
jeisenbe 1296b00
Reduce text-dy to 4 for cranes
jeisenbe 88d8364
Revert marker-transform and text-dy change for cranes
jeisenbe 21d068a
Remove crane rendering from z14
jeisenbe 8f922ed
Render cranes at z16 if height >50 or at z17
jeisenbe 7356649
Merge 'upstream/master' into crane
jeisenbe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -552,6 +552,19 @@ | |
} | ||
} | ||
|
||
[feature = 'man_made_crane'] { | ||
[zoom >= 14][height >= 100], | ||
[zoom >= 15][height >= 50], | ||
[zoom >= 16][height >= 30], | ||
[zoom >= 17] { | ||
marker-file: url('symbols/man_made/crane.svg'); | ||
marker-fill: @man-made-icon; | ||
marker-placement: interior; | ||
marker-clip: false; | ||
marker-transform: 'translate(0,-6)'; | ||
} | ||
} | ||
|
||
[feature = 'man_made_telescope']["telescope:type" != 'optical']["telescope:type" != null] { | ||
[zoom >= 13]["telescope:diameter" >= 60], | ||
[zoom >= 14]["telescope:diameter" >= 30], | ||
|
@@ -564,16 +577,16 @@ | |
} | ||
} | ||
|
||
[feature = 'man_made_telescope']["telescope:type" = 'optical'], | ||
[feature = 'man_made_telescope']["telescope:type" = null], { | ||
[zoom >= 14]["telescope:diameter" >= 8], | ||
[zoom >= 15]["telescope:diameter" >= 4], | ||
[zoom >= 16]["telescope:diameter" >= 2], | ||
[zoom >= 17] { | ||
marker-file: url('symbols/man_made/telescope_dome.svg'); | ||
marker-fill: @man-made-icon; | ||
marker-placement: interior; | ||
marker-clip: false; | ||
[feature = 'man_made_telescope']["telescope:type" = 'optical'], | ||
[feature = 'man_made_telescope']["telescope:type" = null], { | ||
[zoom >= 14]["telescope:diameter" >= 8], | ||
[zoom >= 15]["telescope:diameter" >= 4], | ||
[zoom >= 16]["telescope:diameter" >= 2], | ||
[zoom >= 17] { | ||
marker-file: url('symbols/man_made/telescope_dome.svg'); | ||
marker-fill: @man-made-icon; | ||
marker-placement: interior; | ||
marker-clip: false; | ||
} | ||
} | ||
|
||
|
@@ -1914,6 +1927,7 @@ | |
[feature = 'man_made_telescope'][zoom >= 17], | ||
[feature = 'man_made_water_tower'][zoom >= 17], | ||
[feature = 'man_made_chimney'][zoom >= 17], | ||
[feature = 'man_made_crane'][zoom >= 17], | ||
[feature = 'man_made_waste_water_plant'][zoom >= 17] { | ||
text-name: "[name]"; | ||
text-size: @standard-font-size; | ||
|
@@ -1925,6 +1939,9 @@ | |
[feature = 'historic_wayside_cross'] { | ||
text-dy: 6; | ||
} | ||
[feature = 'man_made_crane'] { | ||
text-dy: 4; | ||
} | ||
[feature = 'historic_city_gate'], | ||
[feature = 'man_made_mast'], | ||
[feature = 'man_made_tower'], | ||
|
@@ -3163,4 +3180,4 @@ | |
trunk/marker-height: 6; | ||
} | ||
} | ||
} | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. End of line mark was deleted here, please get it back. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This whole section (telescope indentation) does not belong here.