-
Notifications
You must be signed in to change notification settings - Fork 309
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
設定ダイアログのリデザイン #2276
設定ダイアログのリデザイン #2276
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
全体的に柔らかい感じになっていて、良い感じだと思いました!!
ちょっとまだコードが追えてないのですが、実際に表示していくつか気になった点をコメントしました!
(気になる点を分けてディスカッションしやすいよう、適当なファイルにコメントさせていただきました 🙇 )
:options="availableAudioOutputDevices" | ||
class="col-7" | ||
</ToggleCell> | ||
<QSlideTransition> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(ただのコメントです)
あ、ここquasar成分残ってそうですね!
デザインの方針の参考になればと思ってちょっとコメントまで。
ぶっちゃけアニメーションするのはこだわりなくて、ONのときに現れたということが分かりやすいようにしたかった感じです。
クリックする対象と現れる対象を見た目的に親子構造にすると良さそうなのですが、たしか離れてるところに現れるやつもあるので親子構造とかには今はできなそう。。。 😇
SelectCellの作成&RowCard全体を押せるようにするのと、項目がdisabledだったときの動作を修正しました。 |
ホバー・クリック時の色を変更するアニメーションを削除してみました。 |
個人的な好みだけで言うと、アニメーションがあるとテンションが上がって作業が進むのでアニメーションは必ずしも消さなくてもいいと思います。 |
個人的には少なくとも問題にはならなさそうには感じました!! もしかしたらSelectが開かなくなってるかも・・・?(勘違いだったらすみません 🙇 ) |
こっちだと普通に開けてそうです! |
@@ -0,0 +1,114 @@ | |||
<template> | |||
<SelectRoot v-model="model" v-model:open="open" :defaultValue :disabled> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
めちゃめちゃ細かいのですが不具合っぽい挙動を発見しました!!
でもなんかradix-vue側の課題な気もしますが。
Selectを開く→画面内の適当なとこをクリックしてSelectを閉じる→もう一度Selectを開く→マウスクリックを離したタイミングで閉じてしまう
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
こっちだと普通に開けてそうです!
すみませんでした、開けました!!!
長いレビューに付き合ってくださってありがとうございます!!!
あとはSelectArrowの部分でワーニングが出てる件だけ変更して完成かなと思います!!!
デザイン的にもコード的にも良い感じになったと感じます!!
不要なクラスの削除もありがとうございます!!
あ、色のアニメーションですが、テーマ変更時に妙なことにならなければ設定されていても全く問題ないと思います・・・!
(アニメーションをなくす方に倒してほしい、という押し付けになってしまっていたら申し訳ないなと思ってのコメントです 🙇 )
全然押し付けとは感じてなかったです…!ホバー時などの色の変化へのアニメーションは、あるにしても柔らかくモダンな印象が得られてモチベになったり、ないにしてもサクサク感があり制作作業に快適性が得られるというように両方にメリットがあると考えていて、後はVOICEVOX的にどちらに倒すかだけだと思ってます。 まあでも、もう少しアニメーションの時間を短くしたり、イージングの仕方を変えるなどが最適な案だという結論に最終的に至るかもしれないので、変更を急いでこのPRにひとまとめにせずいずれ検討する形でもいいのかもですね…! |
失礼しました、ありがとうございます 🙇 🙇 🙇
確かにです! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あっテスト落ちてますね・・・! |
ありがとうございます!お願いします…! |
… into pr/takusea/2276-3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ちょっと手間取ってコミット数が増えてしまってすみません!
マージします!!
(以下ちょっとメモ)
設定ダイアログはスクリーンショットテストをしてるんですが、マウスホバーが入って入ってないのがあるかも。
77898f1
(#2276)
ホバーが入ってない方はセルにマウスカーソルが当たってないだけなのか、はたまたホバー判定が入る前にスクリーンショットが取られたのかどうか不明
もし後者だったらスクリーンショットが撮られるタイミングによって画像が変わってしまってテストが落ちるかも。
もしスクショテストが落ちたらここを疑う。
内容
設定ダイアログのリデザインを行います。コンポーネントごとの変更内容は以下のとおりです。
BaseToggleGroup・BaseToggleGroupItem
BaseRowCard
BaseSelect・BaseSelectItem
BaseCheckbox
ButtonToggleCell・EditButtonCell・ToggleCell
BaseCell
SettingDialog
スクリーンショット・動画など