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

Fix #1229 #1289

Merged
merged 254 commits into from
Apr 16, 2023
Merged

Fix #1229 #1289

merged 254 commits into from
Apr 16, 2023

Conversation

madosuki
Copy link
Contributor

内容

初回起動時と追加キャラクター検出された場合に起動シーケンス内で「並び替えダイアログ」が開かれるが、開かれている間にも起動シーケンス処理が進んでいき、その際に並び替えが反映される前に最初のAudioCellが作られるため初回起動時では四国めたん、追加キャラクター検出時には追加キャラクターが選択された状態になっていました。

store.state.userCharacterOrderをwatchして並び替えが反映されたタイミングでaudioCell.updateFirstCharacterを実行し、その後unwatchするようにしました。

関連 Issue

close #1229

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

2023-04-13.20-21-31.mp4

その他

@madosuki madosuki requested a review from a team as a code owner April 15, 2023 04:34
@madosuki madosuki requested review from Hiroshiba and removed request for a team April 15, 2023 04:34
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

いい感じに思いました!
ちょっといくつかコメントしてみました。

src/components/AudioCell.vue Outdated Show resolved Hide resolved
src/views/EditorHome.vue Outdated Show resolved Hide resolved
src/views/EditorHome.vue Outdated Show resolved Hide resolved
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

ほぼLGTMです!! 変更ありがとうございます!!

src/views/EditorHome.vue Outdated Show resolved Hide resolved
src/views/EditorHome.vue Outdated Show resolved Hide resolved
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!

変更が1ファイルになってシンプルなPRになったなと思いました!
修正ありがとうございました!!!

@Hiroshiba
Copy link
Member

問題ないと思うのでマージします!

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