Theme: move hard-coded colours into tokens, prepare a light or high-contrast variant
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Start with Theme/Tokens.axaml and Theme/Styles.axaml, then use the file counts and colour values in the issue to locate hard-coded colours across the listed views. Compare each value with existing tokens and move the colours into tokens while preserving their rendered values. Check that views use the appropriate resources and compare before and after renders for pixel identity; the issue says the light or high-contrast variant and setting are separate steps.
索引モデルが issue の本文から書いたものです。
説明
Split out of #605, whose wording half shipped on its own (#678, #679).
Problem
App.axaml:5hard-locksRequestedThemeVariant="Dark".Theme/Tokens.axamlis one flatResourceDictionarywith noThemeDictionaries: 25Colors and 26 brushes.- Views hard-code hex colours instead of using the tokens.
Direct sun or glare in a vehicle calls for a light or high-contrast variant. Today that is not possible without touching every view.
Survey (main at 95a6845)
There are 114 hard-coded colour values in .axaml outside Tokens.axaml. That count excludes #123 issue references in comments.
| File | Count |
|---|---|
Theme/Styles.axaml |
39 |
Views/CoMessprotokollView.axaml |
15 |
Views/MasterDataEditorView.axaml |
12 |
Views/EtbView.axaml |
7 |
Views/RolesView.axaml |
6 |
Views/TasksView.axaml, Views/ForcesView.axaml |
4 each |
ScbaView, HomeView, FilesView |
3 each |
| 12 more views | 1–2 each |
The most common values:
| Value | Uses | Token with the same value? |
|---|---|---|
#0F141D |
25 | none (closest: AppBgColor2 #10141C) |
#0C1119 |
25 | none |
#241608 |
10 | AmberWashColor |
#F87171 |
8 | ErrorColor |
#FFFFFF |
7 | none |
#B3070A10 |
7 | none (scrim) |
#2A1013 |
3 | ErrorWashColor |
#FF0000, #92D050, #FFC000 |
3 / 3 / 1 | none (CO-Messprotokoll status colours) |
C# has hex colours only for the CO-Messprotokoll:
CoMessprotokollViewModel.csuses the status colours as string literals.Documents/Sections/CoMessprotokollSection.csduplicates the Amber, SignalHot and Signal tokens for the PDF. The PDF is not themed, so that part can stay.
Plan
- Tokens first, no visual change. Move every hard-coded colour into
Tokens.axaml, reusing the existing token where the value matches. Give#0F141D/#0C1119(panel and grid backgrounds) and the scrim their own tokens. A before/after render of every view should be pixel-identical. ThemeDictionaries. Move the tokens underDark, and switch views fromStaticResourcetoDynamicResourcewhere a variant has to swap at runtime.- A light or high-contrast variant, plus a setting to pick it. This needs a palette decision, ideally checked in a vehicle in daylight.
Each step can be its own PR.
- 主要言語
- C#
- スター
- 1
- フォーク
- 3
- 平均マージ
- 4時間 34分
- マージ済み PR(30日)
- 182
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CodeForFire/lagebuch のほかの issue
-
bug ux
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
CodeForFire/lagebuch#714 ·
メンテナーはふだん 1 日以内に返信
-
bug ux
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
CodeForFire/lagebuch#713 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
CodeForFire/lagebuch#405 ·
メンテナーはふだん 1 日以内に返信
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
CodeForFire/lagebuch#349 ·
メンテナーはふだん 1 日以内に返信
-
documentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
CodeForFire/lagebuch#348 ·
メンテナーはふだん 1 日以内に返信
CodeForFire/lagebuch の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
KnifeLemon/Filee#363 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
area-Native untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
dotnet/machinelearning#7837 ·
メンテナーはふだん 2 日以内に返信
-
[CI] .trgen-ignore required for several grammars.対応中かも @kaby76 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
antlr/grammars-v4#5073 ·
メンテナーはふだん 1 日以内に返信
-
[Enhancement]: Add a search box to the Time Zone dropdown対応中かも @idumlupinar が今日担当しました。 オープンStatus: New
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
dnnsoftware/Dnn.Platform#7579 ·
メンテナーはふだん 2 日以内に返信