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

ツールバーのカスタマイズ可能なボタンの追加 #649

Merged

Conversation

y-chan
Copy link
Member

@y-chan y-chan commented Jan 20, 2022

内容

#489 の分割PRの最後のPRです。
カスタマイズ可能なボタンを追加します。
また、作業中に見つけた軽微なスタイルの崩壊を修正します。

関連 Issue

close #462

#489
#559
#561
#625

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

スクリーンショット 2022-01-21 2 00 03

その他

#489 で追加していた「再生」ボタン(単体再生)は、実装にあたって面倒だったのと、既に画面上に固定されたボタンが存在することから追加しないことにしました。

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!!

挙動とdisableを持っておく設計、良いですね!!
あとはプロジェクトの保存あたりのボタンも使われそうですが、まあキリがなさそうなので一旦こちらの3つが良いのかなと感じました!

src/components/HeaderBar.vue Outdated Show resolved Hide resolved
@y-chan
Copy link
Member Author

y-chan commented Jan 21, 2022

uiLockを直すついでに、これを最初に実装した当時は実装されていなかった「音声を繋げて書き出し」(一つだけとすべて書き出しは実装していたので、それに合わせて出力系をすべてボタン化した)と、仰った「プロジェクト保存」をボタンとして追加しました。

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!!

@Hiroshiba Hiroshiba merged commit e276e57 into VOICEVOX:main Jan 21, 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.

ツールバーのボタン配置をカスタマイズ可能にする
2 participants