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

一部のデザイン調整を取り入れる [update snapshots] #2014

Merged
merged 2 commits into from
Apr 26, 2024

Conversation

romot-co
Copy link
Contributor

@romot-co romot-co commented Apr 26, 2024

内容

リリース用に一部のデザイン修正要素を取り入れます(必要であれば取り込んでいただけると幸いです)

  • ズームスライダーをQSliderに置き換える(input: rangeデフォルトだったため…)
  • ルーラーグリッドを小節だけでなく拍にも表示
  • ノート選択時の表示を修正
  • 複数入力時の表示を修正
  • プレイヘッドの色を修正
  • 高解像度ディスプレイなどでピッチラインがぼやけていたのを修正

関連 Issue

#1810

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

スクリーンショット 2024-04-26 10 17 58 スクリーンショット 2024-04-26 10 19 41
test-voicevox-rec.mov

その他

@romot-co romot-co requested a review from a team as a code owner April 26, 2024 01:28
@romot-co romot-co requested review from y-chan and removed request for a team April 26, 2024 01:28
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!!!

時間が短い中調整していただきありがとうございます!!
ちょっとこちらでいくつか調整させていただくかもです、意図とずれてしまっていたら申し訳ないです 🙇
(そのときは今後のアプデでまた変更できれば・・・!!)

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);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コントラスト比的にちょい高めに感じたので、補足するかもう少し薄くするかさせていただくかもです! 🙇

感覚測るためにいろいろ調べたのでいろいろスクショ貼ってみます。

SynthV
image

Voisona
image

Voisona
image

CeVIO CS7
image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Hiroshiba
ありがとうございます!このあたり適宜落とせれば!
(別で本式?で定義しているものだとグレーの明度定義したものを用意してダークとライトで分けて使っております
今回ライトとグレー共用+アルファチャンネルで代用してしまっておりますが、60%ぐらいでよさそう)

Copy link
Member

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;
Copy link
Member

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;
Copy link
Member

@Hiroshiba Hiroshiba Apr 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ちょっとアニメーションは一旦短くさせていただくか、コメントアウトさせていただくかもです 🙇 🙇 🙇

というのもVOICEVOX全体でアニメーションをほぼ使ってないのと、ソングの中でも統一されてないので、とりあえず一旦統一感のためにという感じです 🙇

(すごい数のノートを一気に変えたりすると、認知負荷に少しプラスの影響を感じました!)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Hiroshiba
こちら削除いただければ…!
※ 意図としては一括入力の変更後など変更されたことへのアテンション(動くとそっちに注意が向いて影響範囲がわかりやすい)ですが、なくてよさそう・アニメーションなしならそっちに統一のほうがよさそうです

src/components/Sing/SequencerNote.vue Outdated Show resolved Hide resolved
src/components/Sing/SequencerNote.vue Outdated Show resolved Hide resolved
background-color: hsl(33, 100%, 50%);
background-color: rgb(179 241 191);
border-color: #65bd7f;
outline: solid 2px #80d998;
Copy link
Member

@Hiroshiba Hiroshiba Apr 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これは自分がずっと見続けてるからな気がしますが、primary colorと色の差があるように感じました。
と思って実際HSVの値で見てみたらだいぶ近くて驚きました。面白い。
image

ちょっとprimaryカラー寄りにさせていただくかもです 🙇
たぶん彩度と色相を更に寄せる感じになるかなと。
もし変えるなら一気に変わる方が良いかなと思ったので、こちらも一旦・・・ 🙇

選択感出すのが難しかったらこのままにさせていただきます!!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Hiroshiba
こちらも異存ございません!
(カラーテーマの作成が間に合っていないので、テーマできたあと一気に変えた方がよさそうです)

Copy link
Member

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 ]

@Hiroshiba
Copy link
Member

マージします!!ありがとうございます!!!

@Hiroshiba Hiroshiba merged commit b2413cc into VOICEVOX:main Apr 26, 2024
9 checks passed
sevenc-nanashi pushed a commit to sevenc-nanashi/voicevox that referenced this pull request Apr 28, 2024
* 一部のデザイン調整を取り入れる [update snapshots]

* Apply suggestions from code review

---------

Co-authored-by: Romot <user@USERs-MacBook-Pro.local>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
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.

None yet

2 participants