search

phinajs / phina.js

phina.js is game library
http://phinajs.com
MIT License
308 stars 51 forks source link

fix Gauge Reference Error #239

Closed negiwine closed 6 years ago

negiwine commented 6 years ago

globalizeしない場合、init内のGaugeの参照でエラーが出るので参照用の変数を追加しました

ihoronir commented 6 years ago

11行目でphina.ui.Gauge.defaultsを参照させるのはどうでしょうか...?

negiwine commented 6 years ago

コメントありがとうございます！ そちらの方法と迷ったのですがLabelAreaやPathShapeで同様に参照用の変数を追加していたので、こちらの方法を意図したのかもしれないと考えてのことでした。 どちらに合わせるのがいいんでしょうか...

ihoronir commented 6 years ago

そうでしたか...! 確かにPathShapeとかでもそうなってますね。私もどちらがいいのかよくわからないです...

simiraaaa commented 6 years ago

LGTM

phina.define の段階で、 superClassが定義済みの場合は、phina.defineの戻り値を参照できますが、定義されていない場合は、うまくいかないことがあります

phi-jp commented 6 years ago

定義されていない場合は、うまくいかないことがあります

確かにそのリスクはあるね. define の返り値 promise にしようかな...

phi-jp commented 6 years ago

@shioleap @negiwine

そちらの方法と迷ったのですがLabelAreaやPathShapeで同様に参照用の変数を追加していたので、こちらの方法を意図したのかもしれないと考えてのことでした。

すみません, 意図してないですw

11行目でphina.ui.Gauge.defaultsを参照させるのはどうでしょうか...?

こっちのほうが良いですね!

phi-jp commented 6 years ago

@negiwine すみません, @minimo さんがやってくれてるやつがあるので こっち close して https://github.com/phinajs/phina.js/pull/243 をマージしますね!

negiwine commented 6 years ago

@phi-jp 考えすぎでしたw よしなにしていただいてありがとうございます！