Skip to content

[TRACKER] refactor(dashboard): Widgets auf MintYColors ThemeExtension migrieren (PR B) - #10

Closed
Toqsick with Copilot wants to merge 1 commit into
mainfrom
copilot/migrate-dashboard-widgets-to-mintytheme
Closed

Toqsick with Copilot wants to merge 1 commit into
mainfrom
copilot/migrate-dashboard-widgets-to-mintytheme

Conversation

Copilot AI commented Aug 20, 2026 •

Copy link
Copy Markdown

🎯 Tracking-PR: Dashboard-Migration auf MintYColors ThemeExtension

⚠️ Hinweis: Der ursprüngliche Copilot-Agent-Job ist fehlgeschlagen (AI-Credits erschöpft). Da Issues im Repo deaktiviert sind, dient dieser Draft-PR als Tracker. Die Umsetzung erfolgt manuell – Branch feature/dashboard-migration oder direkt hier weiterarbeiten.

Anleitung: docs/design/theme-extension-migration.md Abschnitt 3.

Scope (nur diese Dateien)

1. lib/widgets/memory_status.dart

  • Color.fromARGB(255, 70, 153, 221) (CPU) → colors.chartCpu
  • Color.fromARGB(255, 193, 119, 243) (RAM) → colors.chartRam
  • Magic Number cpuLoad < 1 → MintYThresholds.cpuLoadCritical
  • Colors.red (Overload) → colors.statusDanger
  • Zugriff via Theme.of(context).extension<MintYColors>()!

2. lib/widgets/disk_space.dart

  • Magic Number 89 → MintYThresholds.diskUsageWarningPercent
  • Colors.red → colors.statusDanger
  • Color.fromARGB(255, 141, 141, 141) → colors.chartDisk
  • Clean-Icon (Icons.cleaning_services) Verhalten unverändert, Farbe → statusDanger

3. lib/widgets/single_bar_chart.dart ⚠️ erweitert durch Audit-Befund

  • Mutations-Hack entfernen: final-Feld backgroundColor wird aktuell im build() mutiert (Dark Mode → #575757). Komplett ersetzen durch MintYColors.chartTrack (Dark #3A3A3A / Light #D3D3D3)
  • Const-Defaults bleiben als API-kompatible Parameter-Overrides
  • Referenz: docs/design/design-audit-inconsistencies.md §2.1

4. Tests: test/mint_y_tokens_test.dart (neu)

  • MintYColors.dark(MintYAccent.mint): accent 0xff6db443, textDim 0xffb5b5b5
  • MintYColors.light(MintYAccent.debian): accent 0xffd0074e, secondary 0xff4b0523
  • copyWith / lerp korrekt
  • MintYThresholds: 89 / 1.0

Constraints

  • ❌ lib/layouts/mint_y.dart nicht anfassen (Übergangsphase)
  • ❌ Nichts in lib/widgets/hermes/ oder hermes_tokens.dart (Backlog, Branch backlog/hermes-layer)
  • ✅ Keine Behavior-Changes, reiner Token-Swap (Ausnahme: Mutations-Hack-Entfernung)
  • ✅ Widget-APIs rückwärtskompatibel halten

Offener Punkt aus dem Audit

Die visuelle Baseline (v0.7.1-Screenshots, docs/design/visual-baseline-analysis.md §4.2) zeigt 88 % Disk-Auslastung in Amber statt Rot – vor der Migration prüfen, ob der 89 %-Schwellwert im Fork noch greift oder geändert wurde.

Akzeptanzkriterien

  • flutter analyze sauber
  • Tests grün
  • Golden-Baseline (test/goldens/layout_golden_test.dart) lokal unverändert bzw. bewusst aktualisiert

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Toqsick

Toqsick commented Sep 23, 2026

Copy link
Copy Markdown
Owner

Tokens-Thema wird im Fork über den Ausbauplan weiterverfolgt: DS1 Token-Einheit → #65 (in #29 verlinkt).

Quelle: ausbauplan-v0.8.6-v1.0 (Bewertung bestehender Vorhaben).

Toqsick added a commit that referenced this pull request Sep 24, 2026
…ge 2026-09-23

31 Issues auf Toqsick/linux-assistant angelegt (#58–#88, Meilensteine 6–11),
Verknüpfungs-Kommentare auf #25/#26/#27/#28/#29/#30/PR #10 gesetzt,
#31/#32 geschlossen (#28 bleibt offen, Schließkriterium nach WP-S2).
Zuordnungstabelle und Bestands-Aktionen mit echten Nummern gespiegelt.

Co-Authored-By: ZCode <noreply@z.ai>
@Toqsick

Toqsick commented Oct 1, 2026

Copy link
Copy Markdown
Owner

Schließe als not planned: Tracker-PR ohne Code-Änderungen (0 Dateien), Draft vom 2026-08-20 — der zugrunde liegende Copilot-Job war an erschöpften Credits gescheitert.

Das Thema läuft im Fork über den Ausbauplan weiter: DS1 Token-Einheit → #65 (verlinkt aus #29), siehe der bestehende Kommentar oben. Issues sind im Fork inzwischen auch wieder aktiv — der Tracker-PR wird dafür nicht mehr gebraucht; die hier beschriebene Dashboard-Migration auf die MintYColors-ThemeExtension ist an die Ausbauplan-Strecke übergegangen.

Falls die konkreten Widget-Swaps (memory_status / disk_space / single_bar_chart) wieder aufgegriffen werden, gehört das in ein reguläres Issue im Ausbauplan-Kontext.

@Toqsick Toqsick closed this Oct 1, 2026
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.

3 participants