Skip to content

Conversation

@Haruto-Imai
Copy link
Contributor

@Haruto-Imai Haruto-Imai commented Nov 21, 2025

  • ヘルプモーダルを追加しました
  • 僕のPCではなぜか画面下部のボタン配置が狭すぎたので、テンプレートボタンとルール選択ボタンを移動させました
  • ユーザーが、セルの色やセルの誕生/生存条件を編集しやすいよう変更しました
  • このとき、セルをクリックするとセルがboolean型になって盤面が保存できなくなるバグを修正しました
  • コードロード時、ルール選択時、世代数が1になっていたバグを修正し、イベント名を分かりやすくしました
  • isProgress = false; timer = "stopped"; sendEvent("pause"); を isProgress = false; のみにしました
  • これと関連して、再生中にセルを置けるようになりました
  • 速度制限を加えました
  • cellSizeを変更しました
  • life-game.jsを変更したので、コードテンプレートのJSファイルも変更する必要があります(あとでやります)

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 21, 2025

Deploying life-code with  Cloudflare Pages  Cloudflare Pages

Latest commit: cf1e1e6
Status: ✅  Deploy successful!
Preview URL: https://475d5ab5.life-code.pages.dev
Branch Preview URL: https://tutorial-and-help.life-code.pages.dev

View logs

Copy link
Contributor

@aster-void aster-void left a comment

Choose a reason for hiding this comment

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

画面右上のアイコンを、 Life Game 自体の操作に関するものと決めてやると良さそう (たとえばコードリセットなど)

@aster-void
Copy link
Contributor

isProgress は変数名からなにを表す変数なのかが自明でないので、 timer (または timerIsRunning) という名前にしてあげて

@aster-void
Copy link
Contributor

他は良さそう

@Haruto-Imai
Copy link
Contributor Author

画面右上のアイコンを、 Life Game 自体の操作に関するものと決めてやると良さそう (たとえばコードリセットなど)

HelpModalの「画面右上のアイコン」という名称を変更するということですか?
それとも、ボタンの配置を変えるということでしょうか?

@aster-void
Copy link
Contributor

画面右上のアイコンを、 Life Game 自体の操作に関するものと決めてやると良さそう (たとえばコードリセットなど)

HelpModalの「画面右上のアイコン」という名称を変更するということですか? それとも、ボタンの配置を変えるということでしょうか?

これは結果としては両方なんだけど、ボタンの意味でグループ分けをするといいねという意味

なので具体的な修正としてはたとえば「アプリ設定」みたいな名前のグループにして、コードをリセットをコード編集ボタンのところに移動するとか

@Haruto-Imai
Copy link
Contributor Author

コードリセットボタンを画面下部に配置したらまたレイアウトが乱れてしまったので、このままにしようと思います。

@aster-void
Copy link
Contributor

まあ、いいか
よさそう

@Haruto-Imai Haruto-Imai merged commit c86daac into main Nov 21, 2025
4 checks passed
@Haruto-Imai Haruto-Imai deleted the tutorial-and-help branch November 21, 2025 12:34
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.

3 participants