-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
refactor(frontend): remove X theme properties #15376
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #15376 +/- ##
========================================
Coverage 18.66% 18.66%
========================================
Files 766 766
Lines 111337 111337
Branches 1236 1236
========================================
Hits 20783 20783
Misses 89963 89963
Partials 591 591 ☔ View full report in Codecov by Sentry. |
CSSのlight-darkを使うようになると、ダークモードの切り分けをCSSとJSの両方で管理する必要が出てきて後々煩雑になるかもしれない(ブラウザ側のオーバーライド等でこの部分だけが変になるなど)ので、テーマに基準色を置いてCSSのcolor関数等で透明度を調整するほうが良いかもしれない |
ダークモードの切り分けをCSSとJSの両方で管理する必要はどうあがいても絶対に必要(ブラウザ側のUIに適用するにはCSSでブラウザ側にカラースキームを知らせる必要がある)だから諦めるしかなさそう |
それはそうだけど色の決定権がテーマ制御とCSSの記法の両方にあるのがなんか もやった(モヤる程度なので諦めても構わない) |
テーマで全ての色を決定するという設計思想にはなっていないというのがあるわね(それをやろうとするとプロパティが膨大になるし、実装が煩雑になるため、重要度の高い色だけ設定可能にする) |
まあそれはそうか |
Thank you 🙏 |
* refactor(frontend): remove X theme properties * Update MkAutocomplete.vue * Update WidgetCalendar.vue
* refactor(frontend): remove X theme properties * Update MkAutocomplete.vue * Update WidgetCalendar.vue
What
Resolve #12601
Why
リファクタリング
Additional info (optional)
Checklist