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

複数エンジン対応:キャラ立ち絵にエンジン名を追加 #927

Merged
merged 5 commits into from
Sep 17, 2022

Conversation

sevenc-nanashi
Copy link
Member

内容

キャラ立ち絵の左下にエンジン名を表示するようにします。

関連 Issue

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

image
image

エンジンが1つだけの時は表示されません。
image

その他

(なし)

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/components/CharacterPortrait.vue Outdated Show resolved Hide resolved
src/components/CharacterPortrait.vue Outdated Show resolved Hide resolved
@Hiroshiba
Copy link
Member

@raa0121 さんもレビューおねがいできると!!
(いっぱいお願いしてすみません 🙇 )

sevenc-nanashi and others added 2 commits September 8, 2022 18:02
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
@sevenc-nanashi sevenc-nanashi changed the title Add: キャラ立ち絵にエンジン名を追加 複数エンジン対応:キャラ立ち絵にエンジン名を追加 Sep 8, 2022
@Hiroshiba
Copy link
Member

@raa0121 さん、どうでしょう・・・?👀

@raa0121
Copy link
Contributor

raa0121 commented Sep 17, 2022

すいません、今から見ます!

Copy link
Contributor

@raa0121 raa0121 left a comment

Choose a reason for hiding this comment

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

手元で動かせました。問題ないと思います。
遅くなって申し訳ないです。

@Hiroshiba
Copy link
Member

ありがとうございます! マージします!

@Hiroshiba Hiroshiba merged commit 4d8bd36 into VOICEVOX:main Sep 17, 2022
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.

3 participants