-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
全てのタグを追加 #102
全てのタグを追加 #102
Conversation
プルリクありがとうございます!! |
@Shick1112 まず、Text2Frameの仕様の癖をほぼ完璧に理解された上で新たなタグの仕様が書かれていることに感服いたしました。私がこれまでに書いた仕様と一貫性がとれているため、仕様の修正は今の段階ではほぼ必要なさそうです。 一方で、テストケースが存在していません。開発者用ドキュメントがないので伝わっていなかったのかなと思うのですが、実は結構ガチガチにテストケースを書いており、それができたものをmasterブランチに取り込む方針をとっております。そのため、テストケースがない現状では、すぐにmasterブランチにはマージできません。何卒ご理解いただけると幸いです。 以下に、masterに取り込むためのタスクを箇条書きします。
以上のタスクはすべて私が実施しようと思います。一応、私がこのプラグイン開発の言い出しっぺであり、仕様の責任を持っておきたいです。テストケースを作りながら、実装いただいた機能の仕様をじっくりと確認させてください。もちろん、バグがないかのチェックも含んでおります。なにぶん、実装いただいたタグが多いので、1ヶ月以上はかかると思います。コツコツとやらせてください。 ザッと見た限りはなかったのですが、じっくり見た上で仕様変更したほうが良さそうな箇所がありましたら、別途ご相談させてください。黙って仕様および実装をするということはしないと、お約束します。 なにか質問・ご意見等ありましたら、何なりとお申し付けください。よろしくお願いいたします。 |
@HidetoshiKawaguchi @yktsr バグが出た場合はどうしたらいいでしょうか? |
@Shick1112 |
ただいま、ドキュメントをwikiに転機しております。その過程で見つけた、おそらく仕様バグやバグだと思われるものを以下にメモ書きしておきます。見つかり次第追記します。 仕様について議論したいこと
バグ
wikiにヘルプ文を移植する際に、(一貫性のために)文章の構造を変えたタグ
|
issue には議論過程が残っていないが、 選択肢の表示( #91)で同じ問題があり、結局全てのブロックタグ(if や choicesなど場合分けが存在するもの)は現状全て<end>で統一することになっている。ブロックの概念があり、<end>は最も内側のタグと組になるようになっているので、codeが異なっていても<end>で可。 Lines 4845 to 4855 in 7b1d18a
|
差分がわかりにくくなっているので、先行してLinterの修正だけ別のPRで masterへ mergeするのはどうでしょう? |
特に異論ないので、やってください |
おつかれさまです。こちらの方で、ドキュメントをwikiへの移植をしながら、仕様についての詳細なレビューを実施しました。
引数の誤字の修正程度のものや、構文を大きく変更したほうが良さそうなものと、実装の修正労力は大小様々です。 ひとまず、Shick1112さんのご意見をお伺いしたいです。労力をかけて仕様を考え実装していただいたのに恐縮です。しかし、Text2Frameをよりユーザーフレンドリーにするには、私としては一部仕様は修正したほうが良いと考えております。仕様変更をし、コードにも手を入れていくという方針を受け入れてくださるかどうか、お聞かせ願えないでしょうか。 また、もし受け入れてくださる場合、その進め方についてもご相談させてください。Shick1112さんのほうで実装をすることは可能でしょうか。やりたい、やりたくないという観点でも構いません。もし出来ない場合は、私か、@yktsr が担当することが可能です。テストケースについては、私が作成を担当します。テストケースを作っているうちに、更に細かな仕様変更等は発生する可能性があることは、予めご承知いただければ幸いです。 よろしくお願いします。 以下、提案する新しい仕様。通し番号は新しいものなので、現実装の番号とは異なることに注意 音楽再生系のタグ
以下のタグでは、音量やピッチにデフォルト値を設定できるようにする。
例えば、BGMの再生は既にそのような仕様になっているので、それと同様のことを行う。 (30) スキルの増減
操作の指定の`forgot`を`forget`に変更する。
(39) マップのスクロール完了までウェイトを、 マップのスクロールの新仕様「マップのスクロール」は以下のいずれかの記法で組み込むことができます。
向きリスト
速度
完了までウェイトリスト
例1: 下方向の距離100、標準速でマップをスクロール 完了までウェイトしない
例2: 右方向の距離50、1/2倍速でマップをスクロール 完了までウェイト
(46) フキダシアイコンの表示
`userdefined`を`user-defined`に変更する。
ちょっと悩みましたが、一応ツクール準拠にするためにこちらに合わせたいです。
(70) 戦闘の処理以下の3点を変更する
戦闘の処理の新仕様「戦闘の処理」は以下の記法で組み込むことができます。
敵グループは、以下の3種類の指定方法があります。
以下に具体例を示します。 例1: 敵グループID1とエンカウント 逃走不可 敗北不可
例2: 変数ID5の敵グループとエンカウント 逃走可 敗北可
または
例3: ランダムな敵グループとエンカウント 逃走不可 敗北不可
(71) ショップの処理
ショップの処理では、かなりいろいろ仕様を変更する必要があるので、差分は示さずに、以下に新しい仕様をそのまま示します。
ショップの処理の新仕様「ショップの処理」は以下のいずれかの記法で組み込むことができます。
購入のみリスト
商品タイプリスト
価格
例1: アイテムID1を標準価格に設定 購入のみではない
または
例2: 複数の商品を設定 購入のみ
または
(92) 指定位置の情報取得
以下に、新しい仕様をそのまま示します。
指定位置の情報取得の新仕様「指定位置の情報取得」は以下のいずれかの記法で組み込むことができます。
情報タイプリスト
キャラクターで指定する場合のイベントIDリスト
なお、キャラクターで指定するのはツクールMZのみの機能です。 例1: 変数ID1に、現在のマップのX座標10,Y座標20の地形タグの値を格納する
例2: 変数ID2に、現在のマップのX座標を変数4で、Y座標を5で指定しレイヤー1のタイルIDを格納する。
例3: 変数ID3に、このイベントのリージョンIDの値を格納する
(100) 戦闘アニメーションの表示
- MV/MZの切り替えは、プログラム内で行う。引数でどちらを使うかは指定しないものとする
(101) 戦闘行動の強制
インデックスの指定で数値が1ずれているので、それを他のタグと同様に合わせる
|
@HidetoshiKawaguchi ■仕様変更について ■実装について ■通し番号について 以上です。よろしくお願いいたします。 |
@Shick1112 最初に、今後の流れと実装手順についてご説明させていただきます。大きな流れをまず説明すると、以下の通りとなります。
となります。 以上の流れに則ると、まず私がイシューを発行する必要があるので、リポジトリの設定を変えてイシューを発行できるようにしていただけないでしょうか?おそらく、Shick1112さんには画面上のメニューの一番右に「Settings」という項目があり、ここから設定できるかと思います。 イシューを発行できるようになった後に、私のほうでイシューをいくつか発行します。また、それと同時に、それらの仕様を満たしているか確認するためのテストケースを feature/add_all_tagブランチか新たなブランチにプッシュさせていただきます。(Text2Frameは、TDDに則って開発しております) いろいろと書きましたが、ひとまずの依頼事項をまとめると以下のようになります。
これは修正しなくて大丈夫です。実装が終わったあとに、私がText2Frame内のドキュメントに手を入れます。そのときに通し番号も合わせます。 ちょっと込み入った話になっており恐縮ですが、よろしくお願いします。 補足: TDDについてText2Frameは、テスト駆動開発(TDD)に則って開発しております。README.mdの下部に、テストの実行方法が書かれております。
実行には、nodo.jsのmochaというテストフレームワークを使っているので、必要に応じてインストールしてください。なお、テストのコードは、 masterブランチは、原則的に常にこのテストをパスしている状態となっております。 新たな機能の仕様を追加するたびに、テストケースを追加しております。現状本プルリクの新機能にはまだテストケースが存在しないので、それはこれから私が作っていきます。 Shick1112さんが実装されたときは、このnpm run testを実行し、新たに実装している機能のテストがパスされているかを確認していただけると進行がスムーズになるかと思います。(実行が難しくても、こちらで確かめます) |
@Shick1112 Lines 1 to 17 in 7b1d18a
|
@HidetoshiKawaguchi @yktsr eslintとtestを実行してみたところ、エラーが出力されずうまくいっているようでしたが、 以上です。よろしくお願いいたします。 |
@Shick1112 ありがとうございます!イシューを発行できそうなことを確認いたしました。明日より作業に移りますので、しばらくお待ちください。 |
@Shick1112 すみません、イシューは発行できたのですが、追加で以下の権限を私と @yktsr に付与していただけないでしょうか?
五月雨で申し訳ないのですが、よろしくお願いいたします。 |
@HidetoshiKawaguchi 以上です。よろしくお願いいたします。 |
@Shick1112 迅速なご対応ありがとうございます!先程、そちらのリポジトリにてイシューの設定を変え、ブランチもプッシュしました。それぞれのタグの実装については、そちらのリポジトリのイシューやプルリクのコメント欄にて議論しましょう! |
ありがとうございます。一旦自分の方で引き取らせてください。 |
@yktsr |
a675395
to
d6525aa
Compare
Write a help document
@yktsr |
実機での動作確認って取れてます? |
@yktsr とれてます。wikiのmessage.txt(全機能確認用テキスト)をすべてのタグ対応版つくって、ツクールMV/MZの両方で動かして問題なかったです。 |
せっかく全てのタグに対応したので、正面玄関のREADME.mdも説明の順序を少し変えたほうがいいかな、とも思いましたがどうでしょう。あのREADMEは、ver1.0系の、機能が少なかった頃に書かれたもので、今の多機能になった現状では少し読みにくいかもしれません。
ただ、文章を簡単にイベントに取り込む、という中心部分は強力でわかりやすい部分なので、そこはブレない記述が良いと思います。(つまり、前半はそのままに) |
本文のレビューはもう少し時間をください。 |
@yktsr 一旦私が更新してプルリクだすので、その際にチェックしてみてください。
これはそのとおりだと思ってて、前半は変えないでいいと思います。Text2Frameが多くの人に受け入れられたことの80%くらいは、基本機能のメッセージをきれいに取り込めることだと思っています。なので、この基本機能は強調している構成は変えないでいきましょう。
これもそのとおりで、右クリックからのイベント実行にします。wikiのほうはその手順になっているので、それに準拠しましょう。gifじゃなくて、Youtubeからの埋め込み動画が良さそうです。動画撮影も含めて、ちょっと考えてみます。
機能の概要くらいは合ったほうがいいと思います。今のところ、個人的には以下のような変更が良いと思っています。
|
YouTubeはあってもいいですが、再生ボタンを押す手間があるので、一番トップはgifがいいと思います。 |
再生とか停止を押すほうが便利だとは思ったのですが、まぁ詳しい動かし方は文章でもwikiでも説明しているし、ここでの動画は動かし方のイメージを伝えるものと割り切ったということでアニメーションgifのほうが良さそうか |
@yktsr テストケースはイシューに紐付いているブランチにいれてます |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Shick1112 @HidetoshiKawaguchi
長期間に渡っての作業・実装・レビューありがとうございました。
Approveし、2.2.0のリリース候補ブランチへ投入したいと思います。
リリースまで、もう少し作業がありますが、引き継ぎ協力をお願いします。
@yktsr レビューありがとうございます。ひとまず2.2.0に合流でき、ホッとしております。 @Shick1112 ついにyktsr のリポジトリにマージできました。プルリクのご提案から細かい仕様の実装にお付き合いいただきありがとうございます。 2.2.0のリリース(masterへのマージ)は残りの3つを実施した後に行います。これらは私と @yktsr が対応いたします。 引き続き、よろしくお願いいたします。 |
先ほど、修正をmasterに投入し、2.2.0をタグ付け・採番しました。 |
初めまして、Shickと申します。
Text2Frameについて、全てのタグが実装されていたら便利だと思い、
実装されていなかったタグを追加してみました。
お時間があればレビューをしてもらえたら嬉しいです。
特に見てほしいのはタグ名やタグに使用する引数、引数名についてです。
※プログラムの書き方は後々修正してもテキストファイルに影響ないですが、タグ名や引数等は影響があるので
以上です。よろしくお願いします。