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

書き出し先固定時、音声書き出し成功した時にトーストを表示する #1329

Closed
Hiroshiba opened this issue May 31, 2023 · 2 comments · Fixed by #1347
Closed
Labels
優先度:低 初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

内容

書き出し先を固定する機能があります。
これをオンにして音声を書き出しすると、一瞬で音声の保存が完了し、かつ成功したことをフィードバックしていないため、成功したことに気づけない状態になっています。
例:https://twitter.com/shutorabit/status/1663874694934847490

音声書き出しは反復作業となるため、必ず応答が必要になるような成功フィードバック(ダイアログなど)はあまり良くないと思いますが、トーストを表示するのはありかなと思いました。
あとはトーストの右側のボタンとかに二度と表示しないボタンがあれば十分そうだと思います。

Pros 良くなる点

エラーだと勘違いしづらくなる

Cons 悪くなる点

実現方法

ポート自動変更通知トーストが似ていると思います。

$q.notify({
message: `${altPort.from}番ポートが使用中であるため ${engineName} は、${altPort.to}番ポートで起動しました`,
color: "toast",
textColor: "toast-display",
icon: "compare_arrows",
timeout: 5000,
actions: [
{
label: "今後この通知をしない",
textColor: "toast-button-display",
handler: () =>
store.dispatch("SET_CONFIRMED_TIPS", {
confirmedTips: {
...store.state.confirmedTips,
engineStartedOnAltPort: true,
},
}),
},
],
});

その他

@Hiroshiba Hiroshiba added 機能向上 初心者歓迎タスク 初心者にも優しい簡単めなタスク 優先度:低 labels May 31, 2023
@shm11C3
Copy link
Contributor

shm11C3 commented Jun 10, 2023

初めまして。こちらの課題取り組もうと思います!

components/Dialog.ts にトースト呼び出し処理を追記することで実装できると考えていますが、そのような実装で問題ないでしょうか?

@Hiroshiba
Copy link
Member Author

@shm11C3 はじめまして!

なるほどです、とても良いと思います!!
わからないことがあればなんでも聞いてください!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
優先度:低 初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants