We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
現状、ソング側のCSSカラー定義とその利用が扱いづらくなっている 開発時に扱いやすいように修正する
動的変更やシステムカラーの利用についても考慮する
特にカラーに対する相対的な状態変更(ex: アクティブであったりホバーであったり)で Lr値での変更がややこしいため light/darkそれぞれでlr値の変更を容易にするSASS関数/TS関数を追加する
0.2.2?
The text was updated successfully, but these errors were encountered:
なるべく使いやすくて変更が容易なものにしたい
おおまかに:
primary/secondary
surfaceなど
sing-ruler-surface / sing-ruler-measure-line
に分割して相互に疎にする 定義と使用を分離する 各コンポーネントカラーは、そのコンポーネント内で定義・管理する
@function lr($base-lr, $adjustment) { $new-lr: $base-lr + $adjustment; $new-lr: max(0, min($new-lr, 100)); @return '--lr-#{$new-lr}'; } .class { --hover-bg: #{lr($base-lr, 5)}; }
// ホバーでの明度変化量 --hover-state-change: 5; // ホバー変化 @mixin hover-state($base-color, $base-lr) { &:hover { // ライト/ダークモード対応 --hover-l: #{lr($base-lr, if(dark-mode(), $base-lr + var(--hover-state-change), $base-lr - var(--hover-state-change)))}; background-color: oklch( var(--hover-l, l(from var(#{$base-color}))), // 明度 c(from var(#{$base-color})), // 彩度 h(from var(#{$base-color})) // 色相 ); } } .button { @include hover-state(var(--color), 50); }
方法が思いつかない…がコントラスト比などの自動テスト? TypeScriptベースの時はできたもののどうしよう
現状でもCSS変数で外部から注入してプリミティブカラーを変えれば一括で変わる形にはなっている
Sorry, something went wrong.
各コンポーネントカラーをコンポーネント側で定義して使う 便利関数などは必須ではなく特に問題は起こらないはず…?
各コンポーネントカラーをコンポーネント側で定義して使う
そのコンポーネントの中で1回しか出ない色とかは、マストで定義はしなくても良いかも? 逆に何度も使うものは確かに1回定義して使い回した方が絶対良さそう!
romot-co
No branches or pull requests
内容
現状、ソング側のCSSカラー定義とその利用が扱いづらくなっている
開発時に扱いやすいように修正する
動的変更やシステムカラーの利用についても考慮する
Pros 良くなる点
Cons 悪くなる点
実現方法
特にカラーに対する相対的な状態変更(ex: アクティブであったりホバーであったり)で
Lr値での変更がややこしいため
light/darkそれぞれでlr値の変更を容易にするSASS関数/TS関数を追加する
VOICEVOXのバージョン
0.2.2?
OSの種類/ディストリ/バージョン
その他
The text was updated successfully, but these errors were encountered: