Closed Shick1112 closed 9 months ago
プルリクありがとうございます!! 通知が届いておらず気がついていませんでした。レビューさせていただきます! ありがたいことにとても多くの変更をいただいたので、しばしお時間いただきます。
@Shick1112 プルリクありがとうございます。仕様・ドキュメント・テストケースは私が担当なので、その観点でザッとレビューしました。
まず、Text2Frameの仕様の癖をほぼ完璧に理解された上で新たなタグの仕様が書かれていることに感服いたしました。私がこれまでに書いた仕様と一貫性がとれているため、仕様の修正は今の段階ではほぼ必要なさそうです。
一方で、テストケースが存在していません。開発者用ドキュメントがないので伝わっていなかったのかなと思うのですが、実は結構ガチガチにテストケースを書いており、それができたものをmasterブランチに取り込む方針をとっております。そのため、テストケースがない現状では、すぐにmasterブランチにはマージできません。何卒ご理解いただけると幸いです。
以下に、masterに取り込むためのタスクを箇条書きします。
以上のタスクはすべて私が実施しようと思います。一応、私がこのプラグイン開発の言い出しっぺであり、仕様の責任を持っておきたいです。テストケースを作りながら、実装いただいた機能の仕様をじっくりと確認させてください。もちろん、バグがないかのチェックも含んでおります。なにぶん、実装いただいたタグが多いので、1ヶ月以上はかかると思います。コツコツとやらせてください。
ザッと見た限りはなかったのですが、じっくり見た上で仕様変更したほうが良さそうな箇所がありましたら、別途ご相談させてください。黙って仕様および実装をするということはしないと、お約束します。
なにか質問・ご意見等ありましたら、何なりとお申し付けください。よろしくお願いいたします。
@HidetoshiKawaguchi @yktsr レビュー・対応ありがとうございます! 実装したタグが多いので、時間がすごくかかり大変かと思いますが、何卒よろしくお願いいたします。
バグが出た場合はどうしたらいいでしょうか? テストケース等は作成していないのですが、こちらでも実際に使用してみてうまく動かなかった箇所が判明した場合は その都度修正してpush、プルリク等出した方がいいでしょうか?
@Shick1112 お返事ありがとうございます。 バグについて、軽微なものであれば私が直して置きます。 時間がかかりそう or 仕様を変えるレベルになりそうなものが見つかれば、その時はまたご相談させてください。 よろしくおねがいします。
ただいま、ドキュメントをwikiに転機しております。その過程で見つけた、おそらく仕様バグやバグだと思われるものを以下にメモ書きしておきます。見つかり次第追記します。
仕様について議論したいこと
forgot
は正しくはforget
(ツクールの用語に準拠すると)
userdefined
はuser-defined
なのではないだろうか
<IfEnd>
は<End>
と共通でも良いのではないか?
BattleEnd
とする。IfEndでは条件分岐のものと紛らわしいので。購入のみ
だけではないか。Merchandise
と商品
にしたいChangeGetLocationInfo
-> GetLocationInfo
: Changeがついているのは、おそらくバグ?Direct[X座標][Y座標]
WithVariables[X座標の変数ID][Y座標の変数ID]
Character[イベントのID]
バグ
wikiにヘルプ文を移植する際に、(一貫性のために)文章の構造を変えたタグ
戦闘処理の\<IfEnd>は\<End>と共通でも良いのではないか? 調べてみたら、分岐終了と戦闘処理の終了のjsonのcodeは異なっていた。一旦BattleEndとする。IfEndでは条件分岐のものと紛らわしいので。
issue には議論過程が残っていないが、 選択肢の表示( #91)で同じ問題があり、結局全てのブロックタグ(if や choicesなど場合分けが存在するもの)は現状全て\<end>で統一することになっている。ブロックの概念があり、\<end>は最も内側のタグと組になるようになっているので、codeが異なっていても\<end>で可。
差分がわかりにくくなっているので、先行してLinterの修正だけ別のPRで masterへ mergeするのはどうでしょう? 特に異論なければ自分やっちゃいます。
@yktsr
差分がわかりにくくなっているので、先行してLinterの修正だけ別のPRで masterへ mergeするのはどうでしょう? 特に異論なければ自分やっちゃいます。
特に異論ないので、やってください
@Shick1112
おつかれさまです。こちらの方で、ドキュメントをwikiへの移植をしながら、仕様についての詳細なレビューを実施しました。 大方のタグは問題がなかったのですが、以下のいくつかのタグについて、恐縮ですが私としては仕様を変更したいと考えております。
引数の誤字の修正程度のものや、構文を大きく変更したほうが良さそうなものと、実装の修正労力は大小様々です。 私が提案するこれらのタグの新仕様の詳細については、本コメントの下部に記載しています。
ひとまず、Shick1112さんのご意見をお伺いしたいです。労力をかけて仕様を考え実装していただいたのに恐縮です。しかし、Text2Frameをよりユーザーフレンドリーにするには、私としては一部仕様は修正したほうが良いと考えております。仕様変更をし、コードにも手を入れていくという方針を受け入れてくださるかどうか、お聞かせ願えないでしょうか。
また、もし受け入れてくださる場合、その進め方についてもご相談させてください。Shick1112さんのほうで実装をすることは可能でしょうか。やりたい、やりたくないという観点でも構いません。もし出来ない場合は、私か、@yktsr が担当することが可能です。テストケースについては、私が作成を担当します。テストケースを作っているうちに、更に細かな仕様変更等は発生する可能性があることは、予めご承知いただければ幸いです。
よろしくお願いします。
以下、提案する新しい仕様。通し番号は新しいものなので、現実装の番号とは異なることに注意
@HidetoshiKawaguchi 仕様レビューありがとうございます!
■仕様変更について HidetoshiKawaguchiさんが仕様変更を推奨した箇所についてはもちろん修正したほうがいいと思います。 ※私が実装した箇所については仕様が荒かったり、統一されていない箇所があると思うので。
■実装について 私が対応したいと思います。 提案していただいた仕様について不明な点があった場合は相談させてください。
■通し番号について 通し番号についても修正したほうがいいでしょうか? 修正箇所は「対応しているコマンド~」と各タグの使い方の部分の番号と順番になるかと思います。
以上です。よろしくお願いいたします。
@Shick1112 お返事ならびに、仕様変更、さらに実装のお引き受けをいただきありがとうございます!大変嬉しいです。
最初に、今後の流れと実装手順についてご説明させていただきます。大きな流れをまず説明すると、以下の通りとなります。
となります。
以上の流れに則ると、まず私がイシューを発行する必要があるので、リポジトリの設定を変えてイシューを発行できるようにしていただけないでしょうか?おそらく、Shick1112さんには画面上のメニューの一番右に「Settings」という項目があり、ここから設定できるかと思います。
イシューを発行できるようになった後に、私のほうでイシューをいくつか発行します。また、それと同時に、それらの仕様を満たしているか確認するためのテストケースを feature/add_all_tagブランチか新たなブランチにプッシュさせていただきます。(Text2Frameは、TDDに則って開発しております)
いろいろと書きましたが、ひとまずの依頼事項をまとめると以下のようになります。
通し番号についても修正したほうがいいでしょうか?
これは修正しなくて大丈夫です。実装が終わったあとに、私がText2Frame内のドキュメントに手を入れます。そのときに通し番号も合わせます。
ちょっと込み入った話になっており恐縮ですが、よろしくお願いします。 わからないことは、お手数ですがお問い合わせいただけると幸いです。
Text2Frameは、テスト駆動開発(TDD)に則って開発しております。README.mdの下部に、テストの実行方法が書かれております。
npm run test
実行には、nodo.jsのmochaというテストフレームワークを使っているので、必要に応じてインストールしてください。なお、テストのコードは、test/test_json_eq.json
にあります。
masterブランチは、原則的に常にこのテストをパスしている状態となっております。
新たな機能の仕様を追加するたびに、テストケースを追加しております。現状本プルリクの新機能にはまだテストケースが存在しないので、それはこれから私が作っていきます。
Shick1112さんが実装されたときは、このnpm run testを実行し、新たに実装している機能のテストがパスされているかを確認していただけると進行がスムーズになるかと思います。(実行が難しくても、こちらで確かめます)
@Shick1112 私からも一つお願いが。 ファイル全体に何らかのコードフォーマッターがかかっているのですが、それのルールがわかりませんでした。参照先か、または可能であれば、下記eslintファイルの形式でいただけると、コマンドライン or Github Actions で検証できるようになります。
$ npm run lint
@HidetoshiKawaguchi 丁寧な説明ありがとうございます。 一先ずissuesのチェックをオンにしissuesタブが出てくるようになったので、有効化されたと思います。 issuesが有効化されていなかったり、何か別の設定が必要だった場合はお手数ですが再度ご連絡いただけると助かります!
@yktsr 大変申し訳ありません。 私のVSCodeではprettierというコードフォーマッターを使用しており、 私がText2Frameを修正したタイミングで既存のコードフォーマットを崩している箇所がありました(プロパティのダブルクォーテーション等)
eslintとtestを実行してみたところ、エラーが出力されずうまくいっているようでしたが、 既存のコードフォーマットを崩した箇所は元のコードフォーマットに戻した方がいいでしょうか? ※実装の際は再度lint、testは行う予定です。 ・npm run lint ⇒エラーが出力されない ・npm run test ⇒「mocha --inline-diffs test」が実行され、Basic、Crit等の57項目のテストが全てチェックOK
以上です。よろしくお願いいたします。
@Shick1112 ありがとうございます!イシューを発行できそうなことを確認いたしました。明日より作業に移りますので、しばらくお待ちください。
@Shick1112 すみません、イシューは発行できたのですが、追加で以下の権限を私と @yktsr に付与していただけないでしょうか?
五月雨で申し訳ないのですが、よろしくお願いいたします。
@HidetoshiKawaguchi お二方にManage accessのinviteを送りました! 一旦これで各種操作が出来るかどうか試してもらいますでしょうか?
以上です。よろしくお願いいたします。
@Shick1112 迅速なご対応ありがとうございます!先程、そちらのリポジトリにてイシューの設定を変え、ブランチもプッシュしました。それぞれのタグの実装については、そちらのリポジトリのイシューやプルリクのコメント欄にて議論しましょう!
eslintとtestを実行してみたところ、エラーが出力されずうまくいっているようでしたが、 既存のコードフォーマットを崩した箇所は元のコードフォーマットに戻した方がいいでしょうか?
ありがとうございます。一旦自分の方で引き取らせてください。 Google JavaScript Style に合わせて少し調整します。
@yktsr お手数おかけして申し訳ございません。 よろしくお願いいたします。
@Shick1112 おつかれさまです。ひとまず、4つほどテストケースを作成して、PUSHしてみました。 作業を行っているうちに、いろいろと方針について思い至ったことがあるので、共有させてください。
まず、私の仕様の癖とShickさんの仕様の癖の違いが分かってきました。以下の2つです。
Music Note
と Musicnote
この違いなのですが、Text2Frameの使いやすさを考えればどちらかに寄せたほうがいいです。申し訳ないですが、すでに実装されている私の癖に合わせる形で修正したいと考えております。修正後の仕様は、もちろんすべてイシューを発行しご指定させていただきます。何卒ご理解いただけると幸いです。
それに伴い、先に申し上げた仕様変更したいタグの他に、多くのタグで似た修正が必要になってくることが予想されます(まだ全部は確認しておりません)。例えば、2つ以上の単語が結合されている、引数の選択肢を、2つ以上の単語に分割したりであったり、引数のデフォルト値を設定したりすることです。ただし、デフォルト値の設定は、すでに実装されている他のタグと同様のものがあった場合に限ることとします。例えばツクールでよくある「完了までウェイト」のチェックボックスなどがそうです。これは、すでに他のタグでも同様に実装されており、省略した場合はオフという動作になっています。こういうものは、一貫性のために、新たなタグにも追加したいです。ないものについては、なるべく新たなデフォルト値は使わないようにしようと思います。デフォルト値の設定は、あとで追加実装しやすい仕様という理由もあります。
というのが少し中期的な方針になります。ただ、話を撮っ散らかせたくないので、一旦は上述した以下のタグたちのイシューを発行・テストケースの作成を行い、実装しきりたいと思います。
↑↑↑ここまではイシュー発行済↑↑↑
その後に、引数の選択肢の置き換え等、まとめられるものはまとめたりしてイシューを発行していこうと思います。
また、今日までなるべく1日1個以上のイシューを発行しましたが、明日10/20(金)と10/21(土)は私用により作業ができず、またレスポンスが遅くなる可能性が高いです。何卒ご了承ください。
以上です。 なにかご不明点やご意見がありましたら、よろしくお願いいたします。
@HidetoshiKawaguchi お疲れ様です。 私が実装したタグの引数の仕様が統一されておらず、申し訳ありません。 既存に合わせる方針と、一旦は仕様変更があったタグは実装していくことについて理解しました。
また、今日までなるべく1日1個以上のイシューを発行しましたが、明日10/20(金)と10/21(土)は私用により作業ができず、またレスポンスが遅くなる可能性が高いです。何卒ご了承ください。
特に「いつまでに全てのタグを実装したい」、というわけではないので、 HidetoshiKawaguchiさんのご都合の良い時間で進めていただけるだけで嬉しいです! ※作業のほとんどを担当していただいておりますし
以上です。よろしくお願いいたします。
@Shick1112 ご理解いただきありがとうございます。仕様については、私も開発者ドキュメントとか用意しておいて、そこに方針とか書いておけばよかったです。 ペースについても、お言葉に甘えてゆっくりとでも進めさせてください。Shickさんも、ご自身のペースで進めていただければと思います。 引き続きよろしくお願いします。
次のバージョンをversion2.2.0とします。 複数のブランチで作業が進むため、一旦dev-220という作業ブランチにすべての変更を投入し、最終的にdev-220 -> masterに Merge後、tag 2.2.0をつけてリリースとします。 したがって、このPRも一旦dev-220への投入に変更しました。
了解 https://github.com/yktsr/Text2Frame-MV/pull/107 を完了してdev-220にマージ後、Shick1112:feature/add_all_tagにdev-220をマージしてあちらのタグ追加の作業を進めることにします。
ひとまず以下の仕様についてはイシューを発行し、テストケースを作成しました。いくつかはすでに実装され(Shickさん、ありがとうございます)、feature/add_all_tagブランチにマージ済みです。
音楽再生系のタグも含めた他の全タグの仕様確認やテストケースの作成については、今後の進め方を一旦考えるお時間をください。 これまでは1タグ1イシューとしてきましたが、今後は1イシュー複数タグになるかもしれない等があるかもしれません。また、そもそもテストケースを作るだけで終了するものもあるはずですし、そのあたりの工程管理方法を考えます。
これから残りのテストケースを作成していきますが、今後は処理の流れを変更する必要がありそうな処理に限定して、イシューを発行しようと思います。逆を言えば、以下の条件を満たすコマンドについては、イシューを発行せず、私のローカルブランチで修正していき、キリの良いタイミングでfeature/add_all_tag ブランチにマージしていこうと思います。
lasttarget
をLast Target
とする程度のもの以下の表は、原始的ですが工程表です。主に私が、どのテストケースを作成したか、パスしたかを確認するために使うものです。テストケース作成については、ローカルで私がテストケースを作った段階でチェックします。
No | コマンド名 | テストケース作成 | イシュー発行 | テストパス | マージ済み |
---|---|---|---|---|---|
(17) | 所持金の増減 | ✅ | - | ✅ | ✅ |
(18) | アイテムの増減 | ✅ | - | ✅ | ✅ |
(19) | 武器の増減 | ✅ | ✅ | ✅ | ✅ |
(20) | 防具の増減 | ✅ | ✅ | ✅ | ✅ |
(21) | メンバーの入れ替え | ✅ | ✅ | ✅ | ✅ |
(22) | HPの増減 | ✅ | ✅ | ✅ | ✅ |
(23) | MPの増減 | ✅ | - | ✅ | ✅ |
(24) | TPの増減 | ✅ | - | ✅ | ✅ |
(25) | ステートの変更 | ✅ | - | ✅ | ✅ |
(26) | 全回復 | ✅ | - | ✅ | ✅ |
(27) | 経験値の増減 | ✅ | ✅ | ✅ | ✅ |
(28) | レベルの増減 | ✅ | ✅ | ✅ | ✅ |
(29) | 能力値の増減 | ✅ | ✅ | ✅ | ✅ |
(30) | スキルの増減 | ✅ | ✅ | ✅ | ✅ |
(31) | 装備の変更 | ✅ | ✅ | ✅ | ✅ |
(32) | 名前の変更 | ✅ | ✅ | ✅ | ✅ |
(33) | 職業の変更 | ✅ | ✅ | ✅ | ✅ |
(34) | 二つ名の変更 | ✅ | ✅ | ✅ | ✅ |
(35) | プロフィールの変更 | ✅ | ✅ | ✅ | ✅ |
(36) | 場所移動 | ✅ | ✅ | ✅ | ✅ |
(37) | 乗り物の位置設定 | ✅ | ✅ | ✅ | ✅ |
(38) | イベントの位置設定 | ✅ | ✅ | ✅ | ✅ |
(39) | マップのスクロール | ✅ | ✅ | ✅ | ✅ |
(40) | 移動ルートの設定 | ✅ | ✅ | ✅ | ✅ |
(41) | 乗り物の乗降 | ✅ | - | ✅ | ✅ |
(42) | 透明状態の変更 | ✅ | - | ✅ | ✅ |
(43) | 隊列歩行の変更 | ✅ | - | ✅ | ✅ |
(44) | 隊列メンバーの集合 | ✅ | - | ✅ | ✅ |
(45) | アニメーションの表示 | ✅ | ✅ | ✅ | ✅ |
(46) | フキダシアイコンの表示 | ✅ | ✅ | ✅ | ✅ |
(47) | イベントの一時消去 | ✅ | - | ✅ | ✅ |
(56) | 画面の色調変更 | ✅ | ✅ | ✅ | ✅ |
(57) | 画面のフラッシュ | ✅ | ✅ | ✅ | ✅ |
(58) | 画面のシェイク | ✅ | ✅ | ✅ | ✅ |
(59) | 天候の設定 | ✅ | ✅ | ✅ | ✅ |
(69) | ムービーの再生 | ✅ | ✅ | ✅ | ✅ |
(70) | 戦闘の処理 | ✅ | ✅ | ✅ | ✅ |
(71) | ショップの処理 | ✅ | ✅ | ✅ | ✅ |
(72) | 名前入力の処理 | ✅ | - | ✅ | ✅ |
(73) | メニュー画面を開く | ✅ | - | ✅ | ✅ |
(74) | セーブ画面を開く | ✅ | - | ✅ | ✅ |
(75) | ゲームオーバー | ✅ | - | ✅ | ✅ |
(76) | タイトル画面に戻す | ✅ | - | ✅ | ✅ |
(78) | 勝利MEの変更 | ✅ | ✅ | ✅ | ✅ |
(79) | 敗北MEの変更 | ✅ | ✅ | ✅ | ✅ |
(80) | 乗り物BGMの変更 | ✅ | ✅ | ✅ | ✅ |
(81) | セーブ禁止の変更 | ✅ | - | ✅ | ✅ |
(82) | メニュー禁止の変更 | ✅ | - | ✅ | ✅ |
(83) | エンカウント禁止の変更 | ✅ | - | ✅ | ✅ |
(84) | 並び変え禁止の変更 | ✅ | - | ✅ | ✅ |
(85) | ウィンドウカラーの変更 | ✅ | - | ✅ | ✅ |
(86) | アクターの画像変更 | ✅ | ✅ | ✅ | ✅ |
(87) | 乗り物の画像変更 | ✅ | ✅ | ✅ | ✅ |
(88) | マップ名表示の変更 | ✅ | - | ✅ | ✅ |
(89) | タイルセットの変更 | ✅ | - | ✅ | ✅ |
(90) | 戦闘背景の変更 | ✅ | ✅ | ✅ | ✅ |
(91) | 遠景の変更 | ✅ | ✅ | ✅ | ✅ |
(92) | 指定位置の情報取得 | ✅ | ✅ | ✅ | ✅ |
(93) | 敵キャラのHP増減 | ✅ | ✅ | ✅ | ✅ |
(94) | 敵キャラのMP増減 | ✅ | - | ✅ | ✅ |
(95) | 敵キャラのTP増減 | ✅ | - | ✅ | ✅ |
(96) | 敵キャラのステート変更 | ✅ | - | ✅ | ✅ |
(97) | 敵キャラの全回復 | ✅ | - | ✅ | ✅ |
(98) | 敵キャラの出現 | ✅ | - | ✅ | ✅ |
(99) | 敵キャラの変身 | ✅ | - | ✅ | ✅ |
(100) | 戦闘アニメーションの表示 | ✅ | ✅ | ✅ | ✅ |
(101) | 戦闘行動の強制 | ✅ | ✅ | ✅ | ✅ |
(102) | バトルの中断 | ✅ | - | ✅ | ✅ |
@yktsr @HidetoshiKawaguchi
お疲れ様です。 プラグインコマンド(MZ)のインポートについて相談があります。
【やりたいこと】 プラグインコマンド(MZ)の引数の値が無くてもインポートできるようにしたい。 ※以下の例2でも動作するようにしたい。
例1:(ドキュメントにある例そのまま)
<PCZ: TextPicture, set, テキストピクチャの設定, text[ほげ]>
⇒問題なくインポートできる
例2:(例1から「ほげ」を削除)
<PCZ: TextPicture, set, テキストピクチャの設定, text[]>
⇒SyntaxError Unexpected token u in JSON at position 0
のエラーが発生
【理由】 引数の値が無くても(デフォルトで値が入っていない)動作するプラグインも結構あるみたいで、 Frame2Text(イベント⇒テキスト)でエクスポートする際に引数の値が無いと[]で出力される為、 エクスポート⇒インポートで例2のエラーが発生する。
add_all_tagの各作業でお忙しいとは思いますが、お手すきの際に確認してもらえると幸いです。 ※プラグインコマンド(MZ)のインポートの仕様についてあまり理解していない為、何か勘違いしていたらすいません!
以上です。よろしくお願いいたします。
@Shick1112 ご提案ありがとうございます!たしかに、そういったケースは考えられますね。想定ケースから漏れていたので、追加するべきだと思います。
以下のイシューを発行しました。紐づけたブランチには、修正されたテストケースを入れています。 これまでと同じ要領で、実装していただけないでしょうか。73-plugin-command-mz.txtに1行当該のテストケースが追加されています。 https://github.com/Shick1112/Text2Frame-MV/issues/19
@Shick1112 お疲れ様です。 どなたかに頼むことで平行実施できることを私が抱えてボトルネックになるのも不健全だと考えたので、私が実施しようと考えていた軽微な(と思われる)実装もShickさんにお願いできないでしょうか。以下のイシューです。
https://github.com/Shick1112/Text2Frame-MV/issues/30
今回のは、これまでと違って1タグ1イシューではないです。現在作成できている複数個のタグのテストをまとめてプッシュしています。
また、これまでのペースを考えても、私が実装するよりもShickさんにお任せしたほうが良さそうだと考えています。そのため、これから作成されるテストケースについても、複数個できてまとまりが良くなったらまた同様にプッシュしてお願いしようと考えております。(そうなるのもあと1回か2回くらいかなと推測しています。)
ひとまずはイシューのasigneeもShickさんにしていますが、もし難しそうでしたらその旨お申し付けください。 何卒よろしくお願いいたします。
@HidetoshiKawaguchi
お疲れ様です。 実装について了解しました。 実装中に不明な点や仕様の追加・修正したい箇所等があれば連絡致します。 ※基本的に実装についてはこちらに回してもらって構いません!
以上です。よろしくお願いいたします。
@yktsr 本プルリク、すべてのタスクが完了しました。最後に確認していただき、よければ dev220 ブランチにマージしてください。
実機での動作確認って取れてます?
@yktsr とれてます。wikiのmessage.txt(全機能確認用テキスト)をすべてのタグ対応版つくって、ツクールMV/MZの両方で動かして問題なかったです。
せっかく全てのタグに対応したので、正面玄関のREADME.mdも説明の順序を少し変えたほうがいいかな、とも思いましたがどうでしょう。あのREADMEは、ver1.0系の、機能が少なかった頃に書かれたもので、今の多機能になった現状では少し読みにくいかもしれません。 特に、
ただ、文章を簡単にイベントに取り込む、という中心部分は強力でわかりやすい部分なので、そこはブレない記述が良いと思います。(つまり、前半はそのままに)
本文のレビューはもう少し時間をください。
@yktsr たしかにREADMEもこれを機に一新して良さそうですね。そうしましょう。 https://github.com/yktsr/Text2Frame-MV/issues/95 を、ヘルプドキュメントの完全日本語から拡張して、README.mdの更新も入れることにします。
一旦私が更新してプルリクだすので、その際にチェックしてみてください。
ただ、文章を簡単にイベントに取り込む、という中心部分は強力でわかりやすい部分なので、そこはブレない記述が良いと思います。(つまり、前半はそのままに)
これはそのとおりだと思ってて、前半は変えないでいいと思います。Text2Frameが多くの人に受け入れられたことの80%くらいは、基本機能のメッセージをきれいに取り込めることだと思っています。なので、この基本機能は強調している構成は変えないでいきましょう。
トップのgif動画も少し古くて、右クリックからイベント実行、の例に直したほうがbetterでしょうか。あのgifがわかりやすいかは正直自分ではわからないので。
これもそのとおりで、右クリックからのイベント実行にします。wikiのほうはその手順になっているので、それに準拠しましょう。gifじゃなくて、Youtubeからの埋め込み動画が良さそうです。動画撮影も含めて、ちょっと考えてみます。
「その他の機能」の記載粒度が揃ってないのが気になってます。コモンイベント書き出しとか、MZの細かい記述はwikiにして、タグの早見表か、サンプルの文例を貼ったほうがよくないでしょうか。
機能の概要くらいは合ったほうがいいと思います。今のところ、個人的には以下のような変更が良いと思っています。
YouTubeはあってもいいですが、再生ボタンを押す手間があるので、一番トップはgifがいいと思います。
再生とか停止を押すほうが便利だとは思ったのですが、まぁ詳しい動かし方は文章でもwikiでも説明しているし、ここでの動画は動かし方のイメージを伝えるものと割り切ったということでアニメーションgifのほうが良さそうか
@yktsr すまんが、コードレビューしてもらってマージ後、以下の2つのバグ対応お願いできないですかね?
テストケースはイシューに紐付いているブランチにいれてます
@yktsr レビューありがとうございます。ひとまず2.2.0に合流でき、ホッとしております。
@Shick1112 ついにyktsr のリポジトリにマージできました。プルリクのご提案から細かい仕様の実装にお付き合いいただきありがとうございます。
2.2.0のリリース(masterへのマージ)は残りの3つを実施した後に行います。これらは私と @yktsr が対応いたします。
引き続き、よろしくお願いいたします。
先ほど、修正をmasterに投入し、2.2.0をタグ付け・採番しました。 多くの機能追加、テスト、ドキュメントの整備などなど、ほぼ2ヶ月に渡って作業いただいて、ここまでこれたのもひとえにお二人の作業のおかげで、感謝の念に絶えません。 年内にMergeできたのも驚きの速さだったと思います。 改めて作業ありがとうございました!
初めまして、Shickと申します。 Text2Frameについて、全てのタグが実装されていたら便利だと思い、 実装されていなかったタグを追加してみました。
お時間があればレビューをしてもらえたら嬉しいです。 特に見てほしいのはタグ名やタグに使用する引数、引数名についてです。 ※プログラムの書き方は後々修正してもテキストファイルに影響ないですが、タグ名や引数等は影響があるので
以上です。よろしくお願いします。