Skip to content
This repository was archived by the owner on Aug 8, 2023. It is now read-only.

implement symbol-avoid-edges - #3623

Merged
ansis merged 1 commit into
masterfrom
symbol-avoid-edges
Jan 21, 2016
Merged

ansis merged 1 commit into
masterfrom
symbol-avoid-edges

Conversation

@ansis

@ansis ansis commented Jan 20, 2016

Copy link
Copy Markdown
Contributor

Prevent symbols from colliding with tile edges when symbol-avoid-edges is true.

fixes #3582

render tests: mapbox/mapbox-gl-test-suite@bc0272c
-js pull request: mapbox/mapbox-gl-js#1951

@ansis
ansis force-pushed the symbol-avoid-edges branch 2 times, most recently from d5986f8 to 3d1c303 Compare January 20, 2016 23:32
@ansis

ansis commented Jan 20, 2016

Copy link
Copy Markdown
Contributor Author

@lucaswoj can you review? this should be just like the -js pr

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😬

@lucaswoj

Copy link
Copy Markdown
Contributor

Looks :shipit: to me.

ref #3582

if `symbol-avoid-edges` is true, this prevents symbols from colliding
with tile edges.
@ansis
ansis force-pushed the symbol-avoid-edges branch from 3d1c303 to 0fda834 Compare January 21, 2016 00:10
@ansis
ansis merged commit 0fda834 into master Jan 21, 2016
@ansis ansis removed the in progress label Jan 21, 2016
@ansis ansis mentioned this pull request Jan 21, 2016
@ansis
ansis deleted the symbol-avoid-edges branch February 3, 2016 01:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

be more strict about line symbols that cross tile edges

2 participants