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

エンジン追加の際にセキュリティ上の驚異を紹介し、信頼できるエンジンかを問う #1045

Conversation

Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Dec 15, 2022

内容

セキュリティ上の驚異を紹介し、信頼できるエンジンかを問うというタスクです。

関連 Issue

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

image

その他

文面の方針として次のことに意識しました。

  • わからない人は追加を辞めるように強い表現にする
  • 大丈夫そうかを自己判断できる指標を文面に加える
  • 「~して追加してください」という表現を避ける

Copy link
Member

@sevenc-nanashi sevenc-nanashi left a comment

Choose a reason for hiding this comment

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

LGTM!

$q.dialog({
title: "エンジン追加の確認",
message:
"この操作はコンピュータに損害を与える可能性があります。エンジンの配布元が信頼できない場合は追加しないでください。",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"この操作はコンピュータに損害を与える可能性がありますエンジンの配布元が信頼できない場合は追加しないでください",
"この操作はコンピュータに損害を与える可能性がありますエンジンの配布元が信頼できない場合は追加しないでください",

ちょっと強気にしてみるのもありかも…?

Copy link
Member Author

@Hiroshiba Hiroshiba Dec 15, 2022

Choose a reason for hiding this comment

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

面白いですね!
語調に一貫性があったほうが信頼感が増すと思うので、ここだけ感嘆文にするということは避けた方が良いかなと思いました!(とりあえず今回は)

@Hiroshiba
Copy link
Member Author

レビューありがとうございます!
とりあえずマージしますが、追加で意見があればコメントお待ちしています。

@Hiroshiba Hiroshiba merged commit 82313dc into VOICEVOX:main Dec 15, 2022
@Hiroshiba Hiroshiba deleted the エンジン追加の際に確認ダイアログを表示する branch December 15, 2022 15:53
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