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] ヘルプダイアログのデザインの最終調整 #1905

Conversation

takusea
Copy link
Contributor

@takusea takusea commented Mar 5, 2024

内容

ヘルプダイアログのリデザインの最終調整です。具体的には以下のことを行います。

  • スタイルにブレがあった箇所を統一
  • a要素がfocusされた際のoutlineのデザインを合わせる
  • 見出し1のフォントサイズを少し増加
  • セクションの内容に最大幅と中央揃えを設定

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

image
image

その他

これがマージされた後はmainブランチとディレクトリ等々合わせてマージに持っていくのみになりそうです。

@takusea takusea requested a review from a team as a code owner March 5, 2024 16:25
@takusea takusea requested review from y-chan and removed request for a team March 5, 2024 16:25
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・・・と思ったのですが、もしかしたらドロワーの左側のデザインがもしかしたら意識から抜けてたかもです!
figmaにおけるこちらです:
image

意図的だったら後回しでも良いと大丈夫だと思うのですが、忘れだったらmanマージ前に実装しちゃっても良いのかなと思いました!

@takusea
Copy link
Contributor Author

takusea commented Mar 19, 2024

部分部分で何回もデザイン変更のアップデートが重ならないように、mainへのマージ前にリスト部分も含めて実装するほうが良さそうですね。
次プルリクでそこ実装しようと思います。

@Hiroshiba
Copy link
Member

Hiroshiba commented Mar 20, 2024

次プルリクでそこ実装しようと思います。

承知しました!!

1回全部できたあとに再度詳細にレビューできればと思っています。
ということで、次のPRもマージできたときとかフォルダ構成も合わせたりしたときとかに、もし気になった点あれば相談させていただけると・・・!!! 🙇

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 58f82fe into VOICEVOX:project-redisign-help-dialog Mar 20, 2024
7 checks passed
@takusea takusea deleted the adjust-help-dialog-design 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