-
Notifications
You must be signed in to change notification settings - Fork 309
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
Windowのタイトルにバージョン情報を載せる #599
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
初PRありがとうございます!!!
いくつか気になった点、コメントしてみました...!
また、コード全体として、既存の改行を消さずに残しておいてもらえるとありがたいかなと思います(何かの区切りとして、コードの可読性を上げるために改行をつけている場合が多いので...!)。
Co-authored-by: Yuto Ashida <y-chan@y-chan.dev>
Co-authored-by: Yuto Ashida <y-chan@y-chan.dev>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
良い実装だと思います!
ありがとうございました!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@@ -18,7 +18,8 @@ | |||
{{ | |||
(isEdited ? "*" : "") + | |||
(projectName !== undefined ? projectName + " - " : "") + | |||
"VOICEVOX" | |||
"VOICEVOX" + | |||
(currentVersion ? " - Ver. " + currentVersion : "") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
タイトルバーはプロジェクト名も入るので出来るだけ短い方がいいかな(Ver. 0.9.3
-> v0.9.3
)と思いましたが、4文字くらいならそこまで変わらないかなとも思うのでこのままでも良さそうです。
内容
windowのタイトルにバージョン情報を載せます。
関連 Issue
ref #581
スクリーンショット・動画など