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

goolgemapウィジェットの不具合対応について #189

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

noka
Copy link
Contributor

@noka noka commented Jun 10, 2023

よくよく検証したところ、影響は、mootools.jsによるものでした。
https://developers.google.com/maps/documentation/javascript/best-practices?hl=ja
なので、既存のwidgetsを入れ替えずとも、mootools.jsの読み込みを停止すると、一応は動くようです。
廃止関数は使わない方が良いので、書き方を変えたり、関数名の重複などの影響を疑って書き換えたりしたのですが・・・。

こちらで検証した限りでは、完全に動かないAPIが v3.52からで、API v3.51を指定している分にはmootoolsを削除しなくても動くようです(3ヶ月単位でアップデートされるようなので年内には3.51も利用できなくなるような気がします)。

ということで、根本的には、mootools.jsを削除するのがよさそうなのですが、他への影響がわからないため、ご判断いただけますでしょうか?

@yama
Copy link
Member

yama commented Dec 10, 2023

@noka
確認が遅くなってすみません。
mootools.jsですが、何年か前に削除しようとしたのですが、影響があって削除できなかったことを覚えてます。最近は情報が充実しているので、今なら対応方法を調べて削除できるかもです。今度調べてみますね

@yama
Copy link
Member

yama commented Aug 4, 2024

@noka すみません、まだ影響を調べることができてないです。mootools.jsは以前に削除しようとしたのですが、やむなく残すことになったのを覚えています。とはいえもう相当古いので、なるべく早く整理します。
とりあえず今回はいったん保留でリリース準備をしたいと思います。

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.

2 participants