Skip to content
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

chore(assets): oldmove マクロの解説を Standard Map に追加 #307

Merged
merged 4 commits into from Nov 15, 2020

Conversation

aokashi
Copy link
Member

@aokashi aokashi commented Nov 10, 2020

Close: #305

$oldmove の解説キャラクターを Standard Map に追加します。

  • 場所は X=50~60 Y=40~50 あたりにあります
  • キャラクターのメッセージは Java 版原作の $oldmap の解説キャラクターを参考にしています
    • WWA と WWA Wing のように、 WWA が全角と半角で混在しているのですが、原作の雰囲気を残すべきか、文字をできる限り詰める方向にするべきなのかまだはっきりしていないです
  • 動作のサンプルとして、逃げるキャラクターを近くに設けています
    • Ver3.02 以前の逃げると Ver3.10 以降の逃げるでは、前者が結構捕まえやすく、小さい空間の中で効果が確かめられるので逃げるキャラクターを置きました

@aokashi aokashi marked this pull request as ready for review November 11, 2020 14:21
@matsuyuki-a
Copy link
Member

matsuyuki-a commented Nov 14, 2020

物体186では、どちらが緩和なのか、どちらが厳しくなるかをわかりやすくするため、$oldmove=1の時に緩和され, $oldmove=0の時に厳しくなるというのを明示的に書いたほうがよさそうです

物体187では、$oldmove=1 としっかり書いたほうがよいです。

@aokashi
Copy link
Member Author

aokashi commented Nov 14, 2020

物体186では、どちらが緩和なのか、どちらが厳しくなるかをわかりやすくするため、$oldmove=1の時に緩和され, $oldmove=0の時に厳しくなるというのを明示的に書いたほうがよさそうです

直しました。 $oldmove=1 になると緩和する、 $oldmove=0 になると厳しくなると表記してもよかったのですが、作成ツールで確認すれば分かるのではないかなと思います。
($oldmap の解説キャラクターもそんな感じでしたが、不親切であれば 0 と 1 で切り替えできますとキャラクターを別に加えるつもりでいます。)

物体187では、$oldmove=1 としっかり書いたほうがよいです。

直しました。
(メッセージの表記を原作のものに合わせるため、 $oldmove=1 ではなく、 $oldmove を1に指定 と表記しています。)

@matsuyuki-a matsuyuki-a self-requested a review November 15, 2020 13:25
@matsuyuki-a
Copy link
Member

物体186

横の逃げているキャラクタの逃げ方が
少し緩和されます。

上の逃げているキャラクタの逃げ方が
少し緩和されます。

に変更でお願いします 🙇

Copy link
Member

@matsuyuki-a matsuyuki-a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

直したらレビュー不要でマージしちゃってOKです

@matsuyuki-a
Copy link
Member

マージしちゃいます

@matsuyuki-a matsuyuki-a merged commit 8d9b833 into develop Nov 15, 2020
@matsuyuki-a matsuyuki-a deleted the chore/oldmove-standard-map branch November 15, 2020 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

oldmove の解説キャラクターを Standard Map に追加
2 participants