Add: BPM・拍子変更機能を追加 #2303
Add: BPM・拍子変更機能を追加 #2303
Conversation
| () => lastTimeSignature.value.measureNumber === currentMeasure.value, | ||
| ); | ||
|
|
||
| const contextMenuHeader = computed(() => `${currentMeasure.value}小節目`); |
There was a problem hiding this comment.
ここは #2306 の形式で出そうかな～って思ってます。
sevenc-nanashi
requested review from
Hiroshiba
and removed request for
a team
[update snapshots]
There was a problem hiding this comment.
挙動だけ確認させていただきました！
一旦大きく気になったところはなさそう･･･！
@romot-co @sigprogramming
もしよかったらちょっと気になる点先にご確認いただけるとありがたいです！！
BPM や拍子が途中変更できるので、データ所持の設計やコンポーネント構造、あとUX周りで設計を大きく変えないといけないところがあればお聞きしたい感じです！
（データの設計はもともと可変で作っていたから変更なしかも･･･？すごい！！！）
|
BPM・拍子変更機能の実装ありがとうございます！ |
[update snapshots]
[update snapshots]
|
直ったと思います > スナップ線 |
| const timeSignaturesWithTicks = tsPositions.value.map((tsPosition, i) => { | ||
| return { | ||
| position: tsPosition, | ||
| timeSignature: props.timeSignatures[i], | ||
| }; | ||
| }); |
There was a problem hiding this comment.
TimeSignature & { position: number }にしても良いかも。 （テンポの方は変換要らなそう）
There was a problem hiding this comment.
横からすみません！
何度もfind(position)されてるので、たぶんMapにするといろいろきれいになりそうだな〜と思いました！
計算コスト的にMapが一番良い気が･･･するかもしれない！
There was a problem hiding this comment.
確かにMapにするときれいになりそうですね！
（positionが必要になる処理が他にも何か所かあるので、TimeSignatureWithTicks型を作るのも良いかも…？）
内容
タイトル通りです。
関連 Issue
（なし）
スクリーンショット・動画など
その他
現時点でもダイアログの挙動自体はStorybookから確認出来ます。