From 681fa40c3c01d49de108d0ed16daac27bdd08bde Mon Sep 17 00:00:00 2001 From: Alan Wizemann Date: Fri, 1 May 2026 13:27:07 +0200 Subject: [PATCH] fix(skills): use ScarfFont token for OFF pill badge MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The disabled-skill row's "OFF" pill used `.font(.system(size: 9, weight: .semibold))`, which the project CLAUDE.md flags as a code smell ("bypass the type scale… is a code smell"). The design system documents `scarfStyle(.captionUppercase)` as the canonical badge font; switching to it picks up the matching tracking + uppercase casing as a bonus. The pin glyph above (`Image(systemName: "pin.fill").font(.system(size: 9))`) is left as-is — that's intentional glyph sizing on an `Image`, which the design rule explicitly excludes. Co-Authored-By: Claude Opus 4.7 (1M context) --- scarf/scarf/Features/Skills/Views/SkillsView.swift | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scarf/scarf/Features/Skills/Views/SkillsView.swift b/scarf/scarf/Features/Skills/Views/SkillsView.swift index 553d7ae..4591ece 100644 --- a/scarf/scarf/Features/Skills/Views/SkillsView.swift +++ b/scarf/scarf/Features/Skills/Views/SkillsView.swift @@ -242,7 +242,7 @@ struct SkillsView: View { } if !skill.enabled { Text("OFF") - .font(.system(size: 9, weight: .semibold)) + .scarfStyle(.captionUppercase) .padding(.horizontal, 4) .padding(.vertical, 1) .background(ScarfColor.backgroundTertiary)