ソング:ループの機能追加

[romot-co]

内容

以下の内容を行います

• ループ範囲をドラッグおよびメニューで追加・削除できるようにする
• ループをON/OFFできるようにする
• ループ設定をプロジェクトに保存し復帰できるようにする
• ミニマップ(トラックリスト)からもループすることを見越す
• storybookにLoopLaneを追加

関連 Issue

ref #2224
close #2224

スクリーンショット・動画など

2025-01-26.20.46.43.mp4

その他

• マージの解決がややこしかったため、別ブランチ・Issueとして作成いたします
• Grid/ValueChanges/Loopなどレーンごとにコンポーネントとして分割(...逆に複雑になった気もしますが、コンテキストメニュー制御などがややこしかったため
• Containerにロジック/Presentationはなるべくステートレスかつ表示のみの責務を目指したつもり

[voicevox-preview-pages]

🚀 プレビュー用ページを作成しました 🚀

• 📝 エディタ
• 📖 Storybook

更新時点でのコミットハッシュ：5cc9979

[Hiroshiba]

コンフリクトの解決とか、不明な点とかあれば何でも聞いてください！！！！

[romot-co]

@Hiroshiba
おつかれさまです！(おてすきで)
テストにつきまして、engineMockのあたりがわかっておらず…どのようにすればよさそうでしょうか…！
https://github.com/VOICEVOX/voicevox/actions/runs/12991557907/job/36229342596?pr=2506

[Hiroshiba]

テストにつきまして、engineMockのあたりがわかっておらず…どのようにすればよさそうでしょうか…！

ちょっと試した感じ、変更前に戻せばよさそうでした！！
勝手ながらコミット追加させていただきます！ 🙏

聞いてくださってありがとうございます！！！　また何でも聞いてください！！！

[romot-co]

storybookに問題出ているのでそちら修正予定

[romot-co]

そのままやるとSequencerRulerのContainer責務重すぎ＆実装グダグダになったので子にそれぞれミニContainer持たせてロジックわける形にしたものの
親Container+各ロジックはcomposableに切り出す+各子コンポーネントはPresentation相当のみ…もありだったかも

[romot-co]

とりあえずStorybookをstore使う形で以前と同様に動作させた

[Hiroshiba]

ちょっとコード眺めさせていただきました！
Container / Presentation / コンポーザブルをどこをどうするかめちゃくちゃ難しいですね！！！！！
どうすればいいか全然思いつかない！！！

ちょっと迷走させてしまう可能性がある気がするのですが、結構思いついたことベースでコメントさせていただくと･･･

• ルーラーの中の表示物をそれぞれContainerにするのは、割りとアリ････な気がする･･･？
  • もちろんコンポーザブルでも良さそうだけど、どっちがいいのか。。。。。
  • いやーーーーーーわっからない。。。。。
• story内でPresentationのみを見たほうが良いか（Containerは含めないほうが良いか）
  • やはり理想的にはPresentationのみ、useStoreなしが良さそうな気はする
  • けどパラメーターが膨大なので本当にそれが正しいのかわからない。。。
• もしかしたら人間が理解しやすいデータをPresentationに渡し、計算はPresentation内で行った方が良い････････？
  • 例えばテンポや拍子の配列や倍率を渡して、Presentation内で座標を求めるとか。。。。
  • ChatGPT君に聴いたところ、計算はContainerでやりましょうと言われました 😇

全然まとまらなかったです。。。
ContainerとPresentationの切れ目というか、Presentation側のI/Oというかを何にすればいいのかが難しい。。。

あ！　stories作るのはすごく良いと思いました！！！
右クリックメニューもコンテナから与えるの思いつかなかったです。なるほどぉ。

ちょっとこの辺りの設計に関して、ルーラーのstoriesを作ったことのある @sevenc-nanashi さん的にはどうでしょう･･････？
（どうと言われてもって感じかもですが。。。。）