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

Feat/#27 note lane flash #29

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Feat/#27 note lane flash #29

wants to merge 7 commits into from

Conversation

frog3141
Copy link
Contributor

ノーツが光る関数を実装

@watanabeRRR
Copy link
Contributor

これ(送ってくれた動画とは違って)自分の側ではキー押しても光らないけど、キー入力を取得するのはまだ?

@frog3141
Copy link
Contributor Author

キー入力はテスト用に付けた機能なのでPushしたファイルにはつけてません
光る関数をキー入力側で呼び出してもらおうと思ってます。
キー入力で光る機能までこっちのクラスで付けた方がいいですか?

@watanabeRRR
Copy link
Contributor

また新しいエフェクトを追加したい時とかの為に光る関数だけにしたってことですね。じゃあ、キー入力を取得するクラスも別に作っといてくれますかか?

@frog3141
Copy link
Contributor Author

いや、判定とキー入力の読み取りはそっちで作るって話(というかもうInputSystemで作ってあるって話)だったので、
そこのキー入力で光る関数を呼び出してもらおうと思ってました
こっちで別口でキー入力部分を作っちゃって大丈夫ですか?

@watanabeRRR
Copy link
Contributor

なるほど。じゃあこのままで良いです。ありがとうございます。

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.

2 participants