Skip to content

Conversation

@Hiroshiba
Copy link
Owner

VOICEVOX#1879

の提案のために作ってみたけど欠点に気づいて引き換えした。
変更が多すぎるのは別に良い。
一方で元の目的の「UIがアニメーションしないようにする」を達成しづらかった。

アクセント句をmerge/splitする際、accentPhraseのidを一緒にするか変えるか、id的にはどっちでも良いはず。
でもUI的には元のidどちらとも違っていたほうが良い。
ということで、なんかおかしい。

UIにとってのkeyなので、理想的にはコンポーネント内でだけkeyを持っている方が良いかも。
ただそうなるとmergeやsplitもコンポーネント内で完結しないといけない。
それはCOMMANDの実装的にまずいかもしれない。

というメモ

P0ngCh4ng and others added 2 commits February 27, 2024 15:13
AccentPhraseを継承したEditorAccentPhraseという型を作成し、EditorAudioQueryのプロパティとした。

AudioDetail.vueファイルの中でaccentPhrases変数の変更のタイミングで割り振るように
@Hiroshiba
Copy link
Owner Author

いや違った、idの解釈の問題だった気がしてきた。
キャッシュとかにも使うことがあるので、「同じ ID だった場合は同じオブジェクトが割り当たってる」ようにするべきな気がする。
だとするとmerge/splitした時にID が同じになるのはよくない、と考えることもできたかも!

だからmerge/split時にキーを変えれば良さそう。

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