JavaScriptとCSSのリファクタリング - Githubissues

kiridaruma / Onset

TRPG向けの簡単なチャット

MIT License

13 stars 1 forks source link

JavaScriptとCSSのリファクタリング #30

Closed ghost closed 7 years ago

ghost commented 7 years ago

現状でも体感でわかるほど重いわけではないのですが、一部パフォーマンスに関わるリファクタリングを行いました。入室時に大量のログを読み込んだ場合などでは体感での違いがあるかもしれません。

Masterブランチを編集してしまった為、変更履歴がおかしなことになってしまっていますが、実際にはこちらはonset.jsとonset.cssしか編集していません。すみません。

変更点

jQueryオブジェクトの作成はDOMツリー検索を含む重い処理なので、一度作成したら変数に保存してなるべく使いまわすことが推奨される。そのように変更。
DOM要素を他のDOM要素に追加する処理は画面を更新するため重いので、ループの中でいちいち追加していくような処理は推奨されない。その為、DocumentFragmentを利用する。
アニメーションはJavaScriptで行うよりもCSSで行ったほうが高速なため、CSSを利用する。

kiridaruma commented 7 years ago

ありがとうございますー次のアップデートで結構内部構造が変わる予定なので、JS部分は後々書き換わると思いますがCSSはそのまま使うと思います Thanks your PR!