Adopt the house theme: hairline chrome, opacity ladder, display face

cheapnud commented

2026-08-17 00:09:30 +00:00

Owner

First of three style-migration passes: full light palette (white cards on an off-white field, drawer stays dark), exact dark-mode opacity ladder, explicit hairline lines, 12px radius, 260px drawer, Bahnschrift/Barlow display face with a self-hosted woff2 fallback, and every hardcoded border swapped to the palette vars.

cheapnud added 1 commit

2026-08-17 00:09:33 +00:00

Adopt the house theme: hairline chrome, opacity ladder, display face

Claude PR Review / AI Code Review (pull_request) Successful in 31s

Details

Build & Test / Build and Test (pull_request) Successful in 2m38s

Details

5fe7bd0ad1

forgejo-actions commented

2026-08-17 00:10:04 +00:00

🤖 Claude AI Code Review

Summary
Theme migration replacing hardcoded RGBA borders with palette vars, adding a light palette, opacity ladder, custom fonts, and layout properties. Mostly clean mechanical work.
Code Quality ⭐⭐⭐⭐☆
Consistent, well-documented centralization of styling into theme vars — good direction.
Issues

🟢 Home.razor — The colored border cases (rgba(255,87,34,0.5), rgba(33,150,243,0.5), rgba(156,39,176,0.5)) remain hardcoded while the neutral fallbacks were moved to vars. Not a bug, but inconsistent with the PR's stated goal ("every hardcoded border swapped to the palette vars").

🟢 app.css — Only font-weight: 600 is declared for Barlow, but the theme applies Barlow to H4/H5/H6 which may render at other weights depending on MudBlazor typography settings. Non-600 headings on non-Windows platforms will synthesize/fallback rather than use the hosted font. Verify heading weights match, or add the needed weight faces.

Verdict: APPROVE

Verify the barlow-600.woff2 binary is a legitimately licensed font file, but nothing blocks merge.

📊 Tokens: 9800 input + 446 output | 💰 Cost: ~$0.0602
Generated by Claude Opus 4.8 via Gitea Actions

## 🤖 Claude AI Code Review 1. **Summary** Theme migration replacing hardcoded RGBA borders with palette vars, adding a light palette, opacity ladder, custom fonts, and layout properties. Mostly clean mechanical work. 2. **Code Quality** ⭐⭐⭐⭐☆ Consistent, well-documented centralization of styling into theme vars — good direction. 3. **Issues** 🟢 Home.razor — The colored border cases (`rgba(255,87,34,0.5)`, `rgba(33,150,243,0.5)`, `rgba(156,39,176,0.5)`) remain hardcoded while the neutral fallbacks were moved to vars. Not a bug, but inconsistent with the PR's stated goal ("every hardcoded border swapped to the palette vars"). 🟢 app.css — Only `font-weight: 600` is declared for Barlow, but the theme applies Barlow to H4/H5/H6 which may render at other weights depending on MudBlazor typography settings. Non-600 headings on non-Windows platforms will synthesize/fallback rather than use the hosted font. Verify heading weights match, or add the needed weight faces. 4. **Verdict**: **APPROVE** Verify the `barlow-600.woff2` binary is a legitimately licensed font file, but nothing blocks merge. --- 📊 **Tokens**: 9800 input + 446 output | 💰 **Cost**: ~$0.0602 *Generated by Claude Opus 4.8 via Gitea Actions*

cheapnud merged commit b17e3370c3 into master

2026-08-17 00:12:26 +00:00

cheapnud deleted branch d/style-theme

2026-08-17 00:12:26 +00:00

cheapnud referenced this pull request from a commit

2026-08-17 00:12:26 +00:00

Adopt the house theme: hairline chrome, opacity ladder, display face (#8)

Rows
Columns

Adopt the house theme: hairline chrome, opacity ladder, display face #8

🤖 Claude AI Code Review