jQueryのバージョンアップ

[nobuhiko]

• yarn install
• yarn run webpack (--watch)

jQueryのバージョンを上げる場合は

• yarn update jquery

[coveralls]

Coverage remained the same at 51.622% when pulling f641dfb on nobuhiko:version-check-js into 0c3d0da on EC-CUBE:master.

[nanasess]

スマホや、 admin テンプレートの eccube.legacy.js はまだ必要でしょうか？

[nobuhiko]

@nanasess コメントアウトすらされてなかったのでそのまま放置しましたが、消しちゃいますか？

[nanasess]

Suggested change

	"repository": "git@github.com:nobuhiko/eccube-2_13.git",
	"repository": "git@github.com:EC-CUBE/eccube-2_13.git",

EC-CUBE/eccube-2_13 にしておいた方が良いかも

[nobuhiko]

直しました

[nanasess]

@nobuhiko 消しちゃっても大丈夫だと思います！

[3tiles]

jQuery3に上げた影響か、

JQMIGRATE: jQuery.fn.change() event shorthand is deprecated

とか

JQMIGRATE: jQuery.fn.bind() is deprecated

みたいな警告が出るようです。

また、スマホだと、

JQMIGRATE: jQuery(window).on('load'...) called after load event occurred

のようなエラーが出るせいか、トップページのおすすめ商品のスライダーが動作したりしなかったりするようです。

[nobuhiko]

@3tiles
ありがとうございます
警告はjquery migrate の機能なので、出ているエラーを見つつポチポチ直すしかないと思います

スマホのトップページのおすすめ商品は実機だと確かに動かないですね
jquery.flickslide.jsをちょっと見直してみます

[3tiles]

一旦戻されるとのことなので、もう関係ないとは思いますが、折角なので（将来的な？）メモとして残しておきます。

スマホだと、商品詳細の商品画像や関連商品画像も表示されたりされなかったりしたようです。

この手の部分は詳しくないので、外しているかもしれませんが、
https://wemo.tech/109
とか参考になりそうな情報があったので貼っておきます。

あと、警告に関しては、
http://myjquery.blog.fc2.com/blog-entry-36.html
とかに情報が良く纏められているようだったのでこちらも貼っておきます。

[nobuhiko]

商品詳細の商品画像や関連商品画像も jquery.flickslide.js が使われている場所なので同じ問題なのですが、jQuery1系なら動くけど3だと特にエラーもなく動かないので直すのがつらそうです
スマホは、jquery mobile 削除、古いプラグインも削除、っていう別の作業が必要かなと思います

[nanasess]

jquery.flickslide.js はもうメンテされていないようなので、 4系と同じ slick とかに変更してしまっても良い気がします

[nanasess]

リポジトリ名変更お願い致します🙏

[nobuhiko]

意図はしてないですが #299 も直っていると思います

[bbkids]

jQueryをnode_modulesから読み込むようにしてしまった影響でしょうか、
IEからのアクセスではjQueryが読み込まれません。
Firefox, Edgeでは問題ないようですが、IEだとNGです。

[nobuhiko]

@bbkids IE11で動かないの確認しました。原因はES6ですね。修正します

[bbkids]

@babel/polyfillを追加 #346
試して見たのですが、まだNGの様です。

[nobuhiko]

@bbkids ありがとうございます、直ったと思います。

[bbkids]

早々の確認＆修正有難う御座います。
取急ぎ、IEからのアクセスでjqueryが読み込まれるの確認できました！
直っております。