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

do not unwrap For shortest Path, unwrap only if going over antimeridian #13460

Closed
wants to merge 1 commit into from
Closed

do not unwrap For shortest Path, unwrap only if going over antimeridian #13460

wants to merge 1 commit into from

Conversation

osana
Copy link
Contributor

@osana osana commented Nov 27, 2018

closes #13186

@osana osana added the Android Mapbox Maps SDK for Android label Nov 28, 2018
@LukasPaczos LukasPaczos added this to the android-v7.0.0-iowaska milestone Nov 28, 2018
@LukasPaczos
Copy link
Contributor

LukasPaczos commented Nov 28, 2018 •
edited
Loading

Would you be able to add a regression test for #13186? Also, wouldn't it be easier to bundle this fix with #13149 and pass the LatLngBoundsUnwrapped object we discussed to the core at all times?

@tobrun tobrun added the beta blocker Blocks the next beta release label Dec 7, 2018
@osana osana removed the beta blocker Blocks the next beta release label Dec 7, 2018
@osana osana closed this Dec 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@LukasPaczos LukasPaczos Awaiting requested review from LukasPaczos

Assignees
No one assigned
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Milestone
android-v7.0.0-iowaska
Development

Successfully merging this pull request may close these issues.

[android] Wrapping LatLngBounds for shortest path results in ambiguous bounding box behavior
3 participants
@osana @LukasPaczos @tobrun