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

シングエディタの編集状態を保存できるようにする #1781

Closed
3 tasks
y-chan opened this issue Jan 28, 2024 · 3 comments · Fixed by #1829
Closed
3 tasks

シングエディタの編集状態を保存できるようにする #1781

y-chan opened this issue Jan 28, 2024 · 3 comments · Fixed by #1829

Comments

@y-chan
Copy link
Member

y-chan commented Jan 28, 2024

内容

現在、シング画面の状態はまだ保存できません。
これを、保存できるようにしたいです。

どう保存するかを考えています。
保存するデータは、基本的にsinging storeが抱えているデータを保存すればよいのですが、プロジェクトファイルをどうするかが課題になりそうです。
2択です。

  1. トークのプロジェクトファイルに統合する(VOICEVOXとしてプロジェクトファイルはvvprojのみにする)
  2. ソング用にプロジェクトファイルを新しく用意する

今のところ、1で進めようかなと思っているのですが、もしかしたら2の方が良いかも...?
1の場合、最後に開いていた画面をプロジェクトファイル内に保存すると、一発でシング画面に飛べて便利かなと思っています。
2の「方がいい」という理由は特に思いついていなくて、エディタが1つになっているのだからプロジェクトファイルも1つの方がユーザーにとってもわかりやすいだろうと思っています。

設計的な課題としては、1の方だと、既存のプロジェクトファイルを拡張する形になりますが、talk前提のプロジェクトファイルだったので、songをつけ足すのは設計的に結構無理くり感があるかもです。
2の方だと、拡張子を考えたり、変更範囲が大きくなりそうだったり(プロジェクトファイルの拡張子を増やせるようにbackgroundまでいじることになりそう)と、設計はあっさりしつつも実装が少し大変な気もしています。

Pros 良くなる点

ソングエディタの状態が保存できる

Cons 悪くなる点

なし

VOICEVOXのバージョン

0.16

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

@y-chan y-chan changed the title シングの編集状態を保存できるようにする シングエディタの編集状態を保存できるようにする Jan 28, 2024
@Hiroshiba
Copy link
Member

Hiroshiba commented Jan 29, 2024

自分も1の、拡張子vvprojを共有する方法が良いかなと思いました!!

保存するのはtalkかsongどっちか1つだけで良いきがしてます。
2つとも保存したいケースが思いつかないので・・・。
いやでも「アプリを閉じるときにプロジェクトを保存するか聞く」辺りの挙動の変更が大変かも・・・。

データ構造ですが、片方1つだけならzodのdiscriminatedUnion使って{type: "talk", TalkProject}{type: "song", SongProject}のどっちかだけが入るようにするのが良さそう。
https://zenn.dev/terrierscript/books/2023-01-typed-zod/viewer/2-4-or-union#discriminatedunion

2つとも保存なら、まあ普通にtalkキーとsongキーを追加して、それぞれTalkProjectSongProjectを持たせるとかでしょうか。

@sevenc-nanashi
Copy link
Member

これって誰か取り組んでいたりしますか?

@y-chan
Copy link
Member Author

y-chan commented Feb 3, 2024

すみません、Assignつけ忘れてました!
#1800 が大方終わったので私の方で取り組み始めています...!

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

Successfully merging a pull request may close this issue.

4 participants
@Hiroshiba @y-chan @sevenc-nanashi and others