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

[project-redisign-help-dialog] ヘルプ画面のナビゲーションバーのリデザイン #1958

Conversation

takusea
Copy link
Contributor

@takusea takusea commented Mar 24, 2024

内容

ヘルプ画面のナビゲーションバーを新しいデザインのものに変更します。

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

image
image

@takusea takusea requested a review from a team as a code owner March 24, 2024 11:32
@takusea takusea requested review from y-chan and removed request for a team March 24, 2024 11:32
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/styles/new-colors.scss Show resolved Hide resolved
@Hiroshiba Hiroshiba merged commit 5e917cf into VOICEVOX:project-redisign-help-dialog Apr 5, 2024
6 of 7 checks passed
@takusea
Copy link
Contributor Author

takusea commented Apr 8, 2024

背景が半透明のすりガラス(グラスモーフィズム?)になっているの、おしゃれなのですが読んで欲しいテキストがある時はちょっと適さないかもと感じました! 内容が表示されているところは白背景になっていて文字は読みやすいのですが、それはそれで協調性がないと感じられなくもないなと。

がっつり読み物コンテンツのあるダイアログでの使用は難しいのかもと思いました。

裏が半分透けることでオーバーレイしている状態であることを示そうという思惑はありますが、確かに必須ではなく可読性優先するならなくしても良さそうです。

ただ、協調性や統一感という面では半透明をなくしても切り替えの際に緑背景(Background色)と白背景(Surface色)が交差してる時点で欠け気味な気がしてきました。

そこで、読み物系の白背景を全幅取らないようにするか、リスト系の箇所も白背景を入れる(又はサイドバーまで全面白)が対策として考えられそうです。

後者の場合、統一性のためツールバーのカスタマイズダイアログ等、似たレイアウトのデザインの背景色も白にすべきかと思います。

読み物系の白背景を全幅取らないようにする場合:

Policy

リスト系の箇所も白背景を入れる場合:

Licence

@Hiroshiba
Copy link
Member

Hiroshiba commented Apr 30, 2024

@takusea すみません遅くなりました!!(もっと素早くお答えできるように心がけたいと思います 🙇 )
どちらもおしゃれで素敵ですね!!

個人的には読み物系が読み物らしくなっていて好きではあるのですが、後者の全白背景がものすごくしっくりきました!
白背景のリスト表示と以前のリスト表示を見比べると、(以前の)ドロワーとコンテンツの背景色が同じ場合、内容物の境界が若干わかりづらくなっていたかもと感じました。
加えてドロワー内のセルを選んだときの統一感が生まれているので、すごくしっくり来たんだと思います。

もしリストとかも白背景にする場合、結構変更量は大きくなりそうでしょうか・・・? 🙇

@takusea
Copy link
Contributor Author

takusea commented Apr 30, 2024

もしリストとかも白背景にする場合、結構変更量は大きくなりそうでしょうか・・・? 🙇

色変数のちょっとしたテコ入れと、ダークモード時の読み物系の箇所の背景色が暗くなるくらいで、そこまで大きくはないかと!
というのも、ダークモード時にクリック可能な要素を下地から浮いている=背景より明るいという表現がしたいのと、リストコンポーネントの用途として中にコントロール要素が入ることがあり、2段階分明度が欲しいという考えです!

ボタンを入れてみての例はこんな感じです

背景現状、リスト要素現状
image

背景現状、リスト要素明るく
image

背景暗く、リスト要素現状
image

それに合わせた読み物系の背景色の変更

背景現状
image

背景暗く
image

@Hiroshiba
Copy link
Member

@takusea なるほどです!
より暗くした背景色の黒さでも特に問題なさそうに感じました!!
比較図ありがとうございます!!

@takusea takusea deleted the redesign-help-dialog-navigationbar branch July 26, 2024 10:48
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