search

Minori-Akizuki / ddntj

playing TRPG online
GNU General Public License v3.0
25 stars 8 forks source link

検討希望: モジュールバンドラの利用 #16

Closed c6h4clch3 closed 7 years ago

c6h4clch3 commented 7 years ago

フロントエンドで利用するJavaScriptファイルについて、 現状の機能のみでしたら数ファイルで十分かと思いますが、 現在Issueとして上がっているような機能や どどんとふで実装されているような機能をこちらでも実装しようと考えた際、 開発規模が膨れ上がることが想定されます。

Webpack/Browserifyなど、 モジュールバンドラで複数のJSファイルを結合したものを読み込むのはどうでしょうか。

Minori-Akizuki commented 7 years ago

Webpackどう使えばいいのかいま必死に勉強しています…… 一回使った時にVueをモジュール化した時の設定がうまくいかなくて挫折したんですよね… ちょっと切り分けからちゃんとやってみますか…

c6h4clch3 commented 7 years ago

一応ざっとWebpack利用とVueコンポーネント化適用でリファクタリングしました。 少々強引だったりコミットログ汚れてたりですが何かご参考になれば。 https://github.com/c6h4clch3/ddntj/tree/develop

Minori-Akizuki commented 7 years ago

おぉぉありがとうございます……! この辺私もちゃんと読みこまないとワケワカメになるので ちょっと理解するのにお時間ください…

Minori-Akizuki commented 7 years ago

理解追いついていないところもありますが大体読みました…なるほど…… 少しずつ取り込んで行きたいと思います。あるいはプルリクいただけるのであればdeveropにとりこまさせて頂きたいです! (人のforkを横取りする方法を知らない……) よろしくおねがいします!

c6h4clch3 commented 7 years ago

一応プルリク出しました。

ちょっとNode.jsの知識が薄いんでバックエンドはかなり強引なことやってるんじゃないかなと思いますが、フロントはうまくコンポーネント化できたかなと思ってます。