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

WIP: 初回起動時にトークとソングのどちらかを選択できるダイアログを表示する #2409

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

jdkfx
Copy link
Contributor

@jdkfx jdkfx commented Dec 10, 2024

内容

  • 初回起動時にトークとソングのどちらを開くか選択してもらうダイアログを表示する
  • 今後、このダイアログを使用して初期設定などを行えるように、ファイル名はDialog/InitialSettingsDialog.vueとした

関連 Issue

close #2011

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

仮で組んでみたダイアログのUI
CSSが分からないのでいい感じのUIを組めない

InitialSettingDialog

2024-12-13.22.24.04.mov

その他

TODO

  • 初回起動時にのみダイアログを表示させたい
    • src/store/type.tsのSetting Store Typesに選択済みor未選択を保存する(興味のあるエディタでもいい→デフォルトで開くエディタとして初期設定が可能)
    • src/store/setting.tssettingStoreで選択済みor未選択について実装
    • → opendEditorを取得すればいいだけだった。
  • 初回起動時に利用規約の同意後、このダイアログに遷移するようにする
    • 初回起動時のE2Eテストを追加する
  • 選択したエディタに遷移する処理を実装する
  • デバッグ用のボタンを消す

@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Dec 10, 2024

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

更新時点でのコミットハッシュ:7fdd64d

Comment on lines 184 to 196
// 初期設定ダイアログ
const isSelectedEditorType = computed(() => store.state.openedEditor);
console.log(isSelectedEditorType.value);
const isInitialSettingsDialogOpenComputed = computed({
get: () => store.state.isInitialSettingsDialogOpen,
set: (val) => {
if (isSelectedEditorType.value) {
void store.actions.SET_DIALOG_OPEN({
isInitialSettingsDialogOpen: val,
});
}
},
});
Copy link
Contributor Author

@jdkfx jdkfx Dec 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

store.state.openedEditorから開いていたエディタを取得し、undefinedの時にSET_DIALOG_OPENでエディタを選択できるダイアログを表示するような処理にしている。

`openedEditor: EditorType | undefined; // undefinedのときはどのエディタを開くか定まっていない`

https://github.com/VOICEVOX/voicevox/blob/main/src/store/type.ts#L1857

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Hiroshiba

初回起動の動作確認はnpm run electron:serveを実行した際、store.state.openedEditorundefinedとなり、このダイアログが開かれるのでそこで確認できます。
すでにどちらかのエディタを開き、再起動した際にこのダイアログが表示されないという挙動をどう確認しようかなと悩んでいます。

Copy link
Member

@Hiroshiba Hiroshiba Dec 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ちょっと問題を把握できないのですがコメントまで!!

openedEditorは前回どっちのエディターが開かれていたかが入っています!
デフォルト値は今talkに設定されているので、これをデフォルトundefiendにする必要がありそうです。

こうすれば、config.jsonがない状態で起動すればこのダイアログが開かれて、1回エディタが開かれた後にもう一度起動すればダイアログが表示されないはず。
ちなみに開発環境の設定ファイルconfig.jsonC:\Users\[ユーザー名]\AppData\Roaming\voicevox-devにあります。

なんですが、ちょっとこの辺のダイアログを開くかどうか周りとかが入り組んでいるので、こちらでちょこっと変更してプルリクエスト送ろうと思います!!

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