Map.panBy and Point.convert API

[blq]

map.panBy specifies that its first argument can be a PointLike object. Though if you pass in an {x, y} literal it fails in camera.js due to the Point.convert(offset).mult() call that assumes that the output from Point.convert is a real Point instance, which is not what Point.convert spec actually says, it will only convert Array to Point, other objects (literals) unchanged.

/Fredrik Blomqvist

[mourner]

Looks like we want to bring this PR in to fix this: mapbox/point-geometry#12

[blq]

Not sure worth opening another issue? But I just noticed a similar API vs implementation discrepancy for LngLatBounds.extend(obj). The API docs claims LngLatLike is handled but doesn't work.

[ryanhamley]

LngLatBounds.extend(obj). The API docs claims LngLatLike is handled but doesn't work.

Hi @blq this discrepancy was fixed in #9293 and released earlier this week as part of 1.9.0

[blq]

@ryanhamley Thanks, should've searched better! But.. I think LngLatBounds still has bugs. I'll post some examples in that thread.