Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix multiple problems #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix multiple problems #10

wants to merge 4 commits into from

Conversation

gpsfl
Copy link

@gpsfl gpsfl commented Oct 1, 2024

This merge request addresses multiple problems I faced using the library

  • Unset interact.js listeners (see Interact listeners should be removed #9) to prevent issues with multiple maps
  • Add option to disable the _downloadTile method, because it creates a lot of http api requests.
  • Add lazy loading for the _panorama object, so it's only created when it's needed. Otherwise it creates costs in the Google API every time the pegman control is added to a map.

I didn't find a minify script in the repository, so I just used an online minifier and pasted it into the .min.js file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant