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

AccentPhrase全てにidを振ろうとした #2

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

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.

2 participants