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.

Sign up for GitHub

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

Jump to bottom

PR#12 Reviewコメント #17

Open

erk5 opened this issue Mar 8, 2016 · 3 comments

Assignees

Contributor

erk5 commented Mar 8, 2016

Clear/Game Overになったあともう一度ゲームできるようにすると良いかも
Clearしたことがちょっとわかりにくいのでくまの動きを変えたりすると良いかも
ちょっとゲームが小さい？あと中央寄せしておいたほうが良い？
stage002.jsの配列は縦位置合わせておいたほうが見やすいと思いますー
stage002.js CoreとMapのサイズ違うけどOK？
stage002.js GAME_STATUSは定数じゃないので大文字じゃない方が良いかも？

Contributor Author

erk5 commented Mar 8, 2016

PR#12マージされちゃってたので気になったところを書いておきました～
エディタ追加するときにでも見ておいていただければと思います。

erk5 assigned toon258

Contributor

toon258 commented Mar 20, 2016

#22 にて対応しました！

Contributor

toon258 commented Mar 20, 2016

stage002.jsの配列は縦位置合わせておいたほうが見やすいと思いますー
こちら対応漏れてました…他の指摘対応とあわせて別途対応します！

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment