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

ダウンロードボタンの近くに利用規約ボタンを追加 #187

Merged
merged 3 commits into from
Feb 20, 2024

Conversation

jdkfx
Copy link
Contributor

@jdkfx jdkfx commented Feb 13, 2024

内容

トップページのダウンロードボタンの下部に利用規約ページに遷移するボタンを追加

関連 Issue

close #181

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

image
スクリーンショット 2024-02-13 11 42 02

その他

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 18, 2024

プルリクエストありがとうございます!!!
blogは結構色々手をつけていきたいのですが、手が足りてなかったのでとても助かります!!!

トップページなのですが、画面サイズによって結構デザインが崩れてしまうことに気づきました 🙇

ファーストビューに入るように書いたCSSがだいぶイマイチなんだと思います・・・。 🙇

CSS を何とかする手もあるのですが、もう一つ根本的なやり方を思いつきました!
↓ここのダウンロードボタンの左に利用規約ボタンを追加するのはどうでしょう・・・?

ちょっとプルリクエストの方向性とは変わってしまうので、微妙でしたらcloseしていただいても・・・ 🙇 🙇 🙇

@jdkfx
Copy link
Contributor Author

jdkfx commented Feb 19, 2024

@Hiroshiba

CSS を何とかする手もあるのですが、もう一つ根本的なやり方を思いつきました!
↓ここのダウンロードボタンの左に利用規約ボタンを追加するのはどうでしょう・・・?

こちらの方法に変更してボタンを設置します。
また、今回のPRでトップページに設置した利用規約のボタンは上記の変更に伴って削除します。

内容としてはほとんど同じなので、クローズまではしなくてもいいと思います。
このまま作業に取り掛かります。

@jdkfx
Copy link
Contributor Author

jdkfx commented Feb 19, 2024

こんな形でダウンロードモーダルに利用規約のボタンを設置してみました。

スクリーンショット 2024-02-19 10 37 05
2024-02-19.10.33.39.mov

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 f0999b7 into VOICEVOX:master Feb 20, 2024
1 check passed
@jdkfx jdkfx deleted the feature/#181_add_terms_button_on_top branch February 21, 2024 03:13
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