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

[WIP] カラー+スタイルの調整 [HCT版] #2164

Draft
wants to merge 40 commits into
base: main
Choose a base branch
from

Conversation

romot-co
Copy link
Contributor

@romot-co romot-co commented Jul 8, 2024

内容

  • 各カラーの定義を調整(MD3のDynamicSchemeを使い生成)
  • 各スタイルの調整

※ 詳細別途追記

関連 Issue

ref #1810
close #1810

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

スクリーンショット 2024-07-08 15 06 57 スクリーンショット 2024-07-08 15 08 03 スクリーンショット 2024-07-08 15 10 54 スクリーンショット 2024-07-08 15 10 25

カスタマイズ:
sourceColor: #00cde1, variant: fidelity, contrast: +0.1, neutral chroma: 0, neutralVariant chroma 8
スクリーンショット 2024-07-08 15 45 07

その他

若干差分が大きいため、いったんプルリクにいたします

@romot-co romot-co requested a review from a team as a code owner July 8, 2024 06:14
@romot-co romot-co requested review from Hiroshiba and removed request for a team July 8, 2024 06:14
@sevenc-nanashi
Copy link
Member

sevenc-nanashi commented Jul 8, 2024

軽く触ってみた感想です。(個人の感想、実際にこれで解決するかは試してないです)

  • ツールバーが窮屈っぽく感じる、特に左右(トークと比べると分かりやすい)

    • キャラ名とスタイルの余白、BPMとかのフィールドの上下の余白はちょっと削れそう?
      image
      image
  • 鍵盤に若干Disabled感がありそう

    • 鍵盤の線をもうちょっと濃くすれば良さそう?
  • デフォルトの太さが若干変わってそう(トーク含め)

  • スタイル書き換えまくるよりradix-vueあたりで書き直した方が幸せになれそう(どうだろう)

    • これやると差分がとんでもなく大きくなりそう
  • pxとかの幅は/styles/variables.scssで書いて計算した方がわかりやすいかも

  • デザインはめちゃくちゃ好き

@romot-co
Copy link
Contributor Author

romot-co commented Jul 8, 2024

@sevenc-nanashi
ありがとうございます!

ツールバーが窮屈っぽく感じる、特に左右
キャラ名とスタイルの余白、BPMとかのフィールドの上下の余白はちょっと削れそう?

こちら修正してみますね!(小さくできるのであればスペースの問題考えてもよさそう

デフォルトの太さが若干変わってそう(トーク含め)

ほんとでした!medium-weightを有効にしたのが問題っぽいので元にあわせます!

スタイル書き換えまくるよりradix-vueあたりで書き直した方が幸せになれそう

Quasar変更だとだいぶきつい感があるので、可能であればRadixにしたい…

pxとかの幅は/styles/variables.scssで書いて計算した方がわかりやすいかも

これはおっしゃるとおりどっかで一元管理したほうがよさそう!

@romot-co romot-co force-pushed the feature/1810_re_color_from_color branch from 704a831 to 2e4c422 Compare July 20, 2024 00:00
@romot-co romot-co marked this pull request as draft August 15, 2024 10:56
@romot-co romot-co changed the title カラー+スタイルの調整 [WIP] カラー+スタイルの調整 (HCT版) Aug 27, 2024
@romot-co romot-co changed the title [WIP] カラー+スタイルの調整 (HCT版) [WIP] カラー+スタイルの調整 [HCT版] Aug 27, 2024
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