Skip to content

Refactor Slim UI pace rendering and fix pace color path#389

Merged
rygel merged 5 commits intodevelopfrom
codex/pace-color-beta7
Mar 20, 2026
Merged

Refactor Slim UI pace rendering and fix pace color path#389
rygel merged 5 commits intodevelopfrom
codex/pace-color-beta7

Conversation

@rygel
Copy link
Copy Markdown
Owner

@rygel rygel commented Mar 20, 2026

Summary

Fixes the Beta Seven pace-color regression where cards stayed yellow by applying pace-aware color logic in the live render path, then refactors the Slim UI rendering architecture to reduce duplicate logic.

Changes

  • Fix MainWindow card color path to use pace-adjusted color indicator when period/reset metadata is present.
  • Ensure period metadata enrichment supports rolling and model-specific windows for synthetic aggregate children.
  • Add centralized pace policy catalog used by both MainWindow and ViewModel paths.
  • Add a single display-catalog pipeline method for main-window usage list construction.
  • Extract imperative provider/sub-provider card rendering from MainWindow into ProviderCardRenderer.
  • Remove duplicate card-render helper methods from MainWindow code-behind.

Why

  • Eliminates split pace logic across multiple paths.
  • Makes rendering behavior consistent and easier to reason about.
  • Reduces bug surface from duplicated UI decision logic.

@github-actions
Copy link
Copy Markdown
Contributor

Build Performance Report

⏱️ Build Time: 12 minutes

📊 Comparison with main branch:

  • Baseline (avg): 10 minutes
  • Current: 12 minutes
  • Change: 📈 20%

✅ Within acceptable range


This is an automated performance check

@github-actions
Copy link
Copy Markdown
Contributor

Build Performance Report

⏱️ Build Time: 12 minutes

📊 Comparison with main branch:

  • Baseline (avg): 10 minutes
  • Current: 12 minutes
  • Change: 📈 20%

✅ Within acceptable range


This is an automated performance check

@github-actions
Copy link
Copy Markdown
Contributor

Build Performance Report

⏱️ Build Time: 12 minutes

📊 Comparison with main branch:

  • Baseline (avg): 10 minutes
  • Current: 12 minutes
  • Change: 📈 20%

✅ Within acceptable range


This is an automated performance check

@rygel rygel merged commit cfe7909 into develop Mar 20, 2026
21 checks passed
@rygel rygel deleted the codex/pace-color-beta7 branch March 20, 2026 19:10
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