Resolve #4891 flyTo() with padding is broken for Globe

[pjamessteven]

Launch Checklist

• Confirm your changes do not include backports from Mapbox projects (unless with compliant license) - if you are not sure about this, please ask!
• Briefly describe the changes in this PR.
• Link to related issues.
• Include before/after visuals or gifs if this PR includes visual changes.
• Write tests for all new functionality.
• Document any changes to public APIs.
• Post benchmark scores.
• Add an entry to CHANGELOG.md under the ## main section.

[pjamessteven]

need to fix tests.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.95%. Comparing base (cc9f170) to head (e42a666).
Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5406   +/-   ##
=======================================
  Coverage   91.95%   91.95%           
=======================================
  Files         282      282           
  Lines       39050    39051    +1     
  Branches     6863     6853   -10     
=======================================
+ Hits        35907    35910    +3     
+ Misses       3016     3013    -3     
- Partials      127      128    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[HarelM]

Can you show how this looks before and after the fix?
Also please add a changelog item.
The code looks good otherwise.

[pjamessteven]

@HarelM
I added an entry to CHANGELOG.md.

Here's the current 5.1.0 behavior (click zoom level one and then alternate between the two padding buttons), you will see the issue):

https://jsbin.com/wihekiq/

And here is a demo of the fix:

https://jsbin.com/jaxukuz/

[HarelM]

Can you please check the fist checkbox in the initial comment? I can't merge without it.

[pjamessteven]

Done!