1253758 - Upgrade MapBox.js library/JavaScript include to latest 2.3.0 version, from our 2.1.5 bundle

Status: VERIFIED FIXED

(www.mozilla.org :: Bedrock, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

I and others have been seeing performance/latency issues with the MapBox tiles on https://www.mozilla.org/en-US/contact/spaces/ and the various office pages.

We're currently on the 2.1.5 version of the MapBox JS include; the latest right now is 2.3.0 - we should consider upgrading, and testing on dev/stage:

https://www.mapbox.com/mapbox.js/api/v2.3.0/

Comment 1

[Alex Gibson [:agibson]]

Other changes we're going to include in this update are as follows:

- Use map constructor instead of loading assets via tileLayer.
- Allow mapbox.js to handle high-res screen detection.
- Remove secondary use of separate tileLayer for labels.

Comment 2

[Alex Gibson [:agibson]]

Attachment: GitHub pull request

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/d68e2f159dbc443beb05189eb7cdafeb543b94f1
[fix bug 1253758] Update Mapbox to v2.3.0

https://github.com/mozilla/bedrock/commit/4b5ebdac9864933f22b31549d7f1152fb01c2d85
Merge pull request #4013 from alexgibson/update-mapbox

[fix bug 1253758] Update Mapbox to v2.3.0

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED on https://www.mozilla.org/en-US/contact/spaces/san-francisco/ and associated pages off /spaces/ - while it's not always the blazing-fast 5 milliseconds I'm about to post a screenshot of (!), it's much faster, and in multiple tests, I haven't seen anything above 4 seconds, total-transfer times.

Thanks!

Comment 5

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Post-fix screenshot, showing much-better performance numbers