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

Bugfix for location widget - load google before location.js #336

Closed
wants to merge 2 commits into from
Closed

Bugfix for location widget - load google before location.js #336

wants to merge 2 commits into from

Conversation

siesqo
Copy link

@siesqo siesqo commented Nov 7, 2012

Google has to be loaded before the location.js script.
Because IE browsers can't handle this properly.

Siesqo added 2 commits November 7, 2012 13:45
…bug)

Google has to be loaded before the location.js script. Because IE
browsers can't handle this properly.
@dieterve
Copy link
Contributor

Can you give some more details? The location.js file is loaded before google.js but is executed when the document is loaded.

@siesqo
Copy link
Author

siesqo commented Nov 26, 2012

"parent::execute()" needs to be loaded after "google.js",
because parent::execute() adds location.js.

And if you don't do this, the location.js will be loaded before google.js (and then IE will give errors, because he can't find google.js)... Other browsers give no errors...

@siesqo
Copy link
Author

siesqo commented Nov 28, 2012

@dieterve
Copy link
Contributor

dieterve commented Dec 4, 2012

This bugfix is added to the upcoming release (3.5).

For the future: add new changelog items at the bottom of the changelog to preserve history.

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.

None yet

2 participants