add: ダイアログのデザインを調整し、文言も調整していく #2410
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ： |
Hiroshiba
commented
Dec 13, 2024
| export type DialogType = "none" | "info" | "error" | "question" | "warning"; | ||
| export type DialogType = "info" | "error" | "question" | "warning"; |
There was a problem hiding this comment.
noneとinfoの役割の違いがややこしかったので削除しました。
infoにするとnone相当（アイコンがつかない）になってます。
There was a problem hiding this comment.
ダークモードのボタンの感じがなんかだいぶ微妙かもだけど････････まあ一旦とりあえずこれで･･････
Hiroshiba
commented
Dec 13, 2024
There was a problem hiding this comment.
@sevenc-nanashi ダイアログのデザイン変えますというご共有まで･･･！
もうちょっとかっこよくしたい気持ちもありますが、まあ一旦･･･。
（あとダイアログのデザイン統一したい･･･。）
sevenc-nanashi
requested changes
Dec 13, 2024
|
レビューありがとうございます！ マージします！ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
ダイアログのデザインを調整し、文言も調整しました。
ボタンの見た目は、とりあえずデフォルトはoutlineに、そうじゃないものはunrelavedにしました。
なんかもうちょっと良い見た目ありそうだけど、まあ一旦ちょっとこれで。。。
良いデザイン入ってほしい･･･。
関連 Issue
スクリーンショット・動画など
見た目こんな感じになりました。
その他
ダイアログの文面ですが、タイトルは疑問文にし、本文には疑問文を含めない方針にしました。
本部に疑問文を含めないのはAdobeのガイドラインやmaterial designのガイドラインに沿い、不要な文面を削除するためです。
タイトルを疑問文にするのはAdobeのガイドラインに反しているのですが、どうしても質問になるダイアログもあるので、統一するなら疑問文の方かなと。
（分かりやすい例だと、保存せずに消そうとした時に「プロジェクトファイルを保存しますか？」と出るやつ。「プロジェクトファイルを保存する」というタイトルだと違和感がある）
ちなみにデザインはちょっと迷ったのですが、全部outlineにするとこうなってしまってprimaryが見づらくなってしまったので、とりあえず。。。
