Issue

jsファイルの圧縮化 #748

wada opened this at 2021-04-20T10:17:44; closed

Description

opened by wada at 2021-04-20T10:17:44; updated 2024-05-29T10:41:56

ファイルが肥大化しているので、ひとまず圧縮してファイルサイズを小さくする
https://firstlayout.net/minify/

Comment

#10442 by wada at 2021-04-20T10:18:01; updated 2021-04-20T10:40:21

d4ac4205bcdebe65a0520735764ea9e7f1591f5d

#composer
cd ./jr-web
composer install

#jsの圧縮(同じディレクトリに圧縮されたjsが出力される)
php minify_js.php public_html/js/JOLLYROGER.js

Comment

#10476 by kouho at 2021-04-21T17:22:23; updated 2021-04-21T17:25:26

@wada
まず使用ツールについて 和田さん提案のもので良いかと思います。
導入がやりやすく圧縮率が他のツールに比べて特に低いわけではないようでしたので。

jr_battle.min.js jr_main.min.js jr_mission.min.js

の3ファイルの修正について動作も確認しました。ページの読み込みもローカル環境上で約0.1秒ほど早くなっているようでした。問題ないと思います。

jsファイルで他に

jr_collect_ef.js ship_strengthening以下のjs island_unlock_js

の3ファイルが100KBを超えていましたのでこちらも圧縮した方が良いかもしれません。

Comment

#10477 by iwamoto at 2021-04-21T17:48:35; updated 2021-04-21T17:48:35

@wada
各ファイルの動作については問題ない思われます。

こちらで確認した際に圧縮前の ``JOLLYROGER.js` と圧縮後 `jr_main.min.js``に差異がありました。
内容としては下記3点のコミットが反映されていませんでした。
ただ、和田さん側で圧縮前のファイルがこれら反映前だったであれば問題ないと思います。

commit bde5c1ec7f5b84ca03701fcbfb63dacacc8d1353 -> カードホルダー、ストック+1獲得時に直接配布表示
commit a6c7820252b47744268326604afa7cca4c8af74d -> ステータス画面に追記
commit 6ebe8387f9025a6083326699529b19900d4d2811 -> 秘宝を連続使用できないようにする

Comment

#10480 by wada at 2021-04-21T19:19:48; updated 2021-04-21T19:19:48

確認ありがとうございます。
このツールを使用してjsファイルの圧縮を行います。
本番環境用のjsファイルは全て圧縮するつもりです(ファイル名が、jr_*.js)

@kouho
コミットされたmin.jsファイルは古いものなので、ファイル反映されてないことが明確であれば問題ないです。

Comment

#10481 by kouho at 2021-04-21T19:23:34; updated 2021-04-21T19:23:34

@wada
了解です。

wada refer at 2021-04-22T19:47:56: 752:jsファイルの圧縮化
wada add_label at 2021-04-22T19:48:56: 改善
kouho refer at 2021-05-13T12:35:56: 800:【5/13】リリース
wada close at 2021-05-17T09:57:04: Close