-
Notifications
You must be signed in to change notification settings - Fork 304
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
一部のデザイン調整を取り入れる [update snapshots] #2014
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.
LGTM!!!
時間が短い中調整していただきありがとうございます!!
ちょっとこちらでいくつか調整させていただくかもです、意図とずれてしまっていたら申し訳ないです 🙇
(そのときは今後のアプデでまた変更できれば・・・!!)
background: colors.$primary; | ||
border-left: 1px solid rgba(colors.$background-rgb, 0.83); | ||
border-right: 1px solid rgba(colors.$background-rgb, 0.83); | ||
background: rgba(colors.$display-rgb, 0.8); |
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.
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.
@Hiroshiba
ありがとうございます!このあたり適宜落とせれば!
(別で本式?で定義しているものだとグレーの明度定義したものを用意してダークとライトで分けて使っております
今回ライトとグレー共用+アルファチャンネルで代用してしまっておりますが、60%ぐらいでよさそう)
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.
なるほどです!!
ちょっと面倒かもですがグレーとmixするとかもありかも・・・?
border-left: 1px solid rgba(colors.$background-rgb, 0.83); | ||
border-right: 1px solid rgba(colors.$background-rgb, 0.83); | ||
background: rgba(colors.$display-rgb, 0.8); | ||
will-change: transform; |
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.
これ初めて見ました。多用すると危ないっぽいんですね!
playheadに付け続けるのは良さそう。
position: absolute; | ||
width: calc(100% + 1px); | ||
height: 100%; | ||
background-color: colors.$primary; | ||
border: 1px solid rgba(colors.$background-rgb, 0.5); | ||
border-radius: 2px; | ||
border-radius: 4px; | ||
transition: all ease-out 0.16s; |
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.
ちょっとアニメーションは一旦短くさせていただくか、コメントアウトさせていただくかもです 🙇 🙇 🙇
というのもVOICEVOX全体でアニメーションをほぼ使ってないのと、ソングの中でも統一されてないので、とりあえず一旦統一感のためにという感じです 🙇
(すごい数のノートを一気に変えたりすると、認知負荷に少しプラスの影響を感じました!)
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.
@Hiroshiba
こちら削除いただければ…!
※ 意図としては一括入力の変更後など変更されたことへのアテンション(動くとそっちに注意が向いて影響範囲がわかりやすい)ですが、なくてよさそう・アニメーションなしならそっちに統一のほうがよさそうです
background-color: hsl(33, 100%, 50%); | ||
background-color: rgb(179 241 191); | ||
border-color: #65bd7f; | ||
outline: solid 2px #80d998; |
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.
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.
@Hiroshiba
こちらも異存ございません!
(カラーテーマの作成が間に合っていないので、テーマできたあと一気に変えた方がよさそうです)
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.
📝 lab色空間で明度ずらす場合
primaryのlabはたぶん[ 80.79626038641231, -22.95295422883703, 14.365116650466048 ]
マージします!!ありがとうございます!!! |
* 一部のデザイン調整を取り入れる [update snapshots] * Apply suggestions from code review --------- Co-authored-by: Romot <user@USERs-MacBook-Pro.local> Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
内容
リリース用に一部のデザイン修正要素を取り入れます(必要であれば取り込んでいただけると幸いです)
関連 Issue
#1810
スクリーンショット・動画など
test-voicevox-rec.mov
その他