fix: replace text-secondary with text-muted-foreground in branching components

In Tailwind v4, text-secondary resolves to --color-secondary (#2e3140),
a dark surface color — NOT --color-text-secondary (#848b9b). This made
all secondary text invisible on dark backgrounds.

The correct class is text-muted-foreground which maps to #848b9b.
This matches the pattern used by existing FlowPilot components.

Also reverts the unnecessary index.css variable bump from prior commit.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
chihlasm
2026-03-24 22:36:01 +00:00
parent 4c42522925
commit 01836d6a2d
6 changed files with 27 additions and 27 deletions

View File

@@ -324,7 +324,7 @@ function MockResolutionPanel({ outputs }: { outputs: ResolutionOutputResponse[]
'flex items-center gap-1.5 px-3 py-2.5 text-xs font-medium transition-colors border-b-2 -mb-px',
isActive
? 'border-accent text-accent-text'
: 'border-transparent text-secondary hover:text-primary hover:border-hover'
: 'border-transparent text-muted-foreground hover:text-primary hover:border-hover'
)}
>
<Icon size={13} />
@@ -371,7 +371,7 @@ function MockResolutionPanel({ outputs }: { outputs: ResolutionOutputResponse[]
<button
type="button"
onClick={handleEditToggle}
className="rounded-[5px] border border-default px-3 py-1.5 text-xs text-secondary hover:bg-elevated hover:text-primary transition-colors"
className="rounded-[5px] border border-default px-3 py-1.5 text-xs text-muted-foreground hover:bg-elevated hover:text-primary transition-colors"
>
Cancel
</button>
@@ -382,7 +382,7 @@ function MockResolutionPanel({ outputs }: { outputs: ResolutionOutputResponse[]
type="button"
onClick={handleEditToggle}
disabled={!activeOutput}
className="flex items-center gap-1.5 rounded-[5px] border border-default px-3 py-1.5 text-xs text-secondary hover:bg-elevated hover:text-primary transition-colors disabled:opacity-40"
className="flex items-center gap-1.5 rounded-[5px] border border-default px-3 py-1.5 text-xs text-muted-foreground hover:bg-elevated hover:text-primary transition-colors disabled:opacity-40"
>
<Pencil size={12} /> Edit
</button>
@@ -392,7 +392,7 @@ function MockResolutionPanel({ outputs }: { outputs: ResolutionOutputResponse[]
disabled={!activeOutput}
className={cn(
'flex items-center gap-1.5 rounded-[5px] border border-default px-3 py-1.5 text-xs transition-colors disabled:opacity-40',
copied ? 'border-success text-success' : 'text-secondary hover:bg-elevated hover:text-primary'
copied ? 'border-success text-success' : 'text-muted-foreground hover:bg-elevated hover:text-primary'
)}
>
{copied ? <Check size={12} /> : <Copy size={12} />}
@@ -402,7 +402,7 @@ function MockResolutionPanel({ outputs }: { outputs: ResolutionOutputResponse[]
type="button"
onClick={() => toast.success('Pushed to PSA (mock)')}
disabled={!activeOutput}
className="flex items-center gap-1.5 rounded-[5px] border border-default px-3 py-1.5 text-xs text-secondary hover:bg-elevated hover:text-primary transition-colors disabled:opacity-40"
className="flex items-center gap-1.5 rounded-[5px] border border-default px-3 py-1.5 text-xs text-muted-foreground hover:bg-elevated hover:text-primary transition-colors disabled:opacity-40"
>
<Send size={12} /> Push to PSA
</button>
@@ -469,7 +469,7 @@ export default function DevBranchingPage() {
toBranch={activeBranch}
/>
) : (
<p className="text-xs text-secondary">Click a branch in the sidebar to see the transition bar.</p>
<p className="text-xs text-muted-foreground">Click a branch in the sidebar to see the transition bar.</p>
)}
</section>
@@ -519,34 +519,34 @@ export default function DevBranchingPage() {
<div className="bg-card border border-default rounded-lg p-4 text-sm">
<div className="grid grid-cols-2 gap-2">
<div>
<span className="text-secondary">Label:</span>{' '}
<span className="text-muted-foreground">Label:</span>{' '}
<span className="text-heading">{activeBranch.label}</span>
</div>
<div>
<span className="text-secondary">Status:</span>{' '}
<span className="text-muted-foreground">Status:</span>{' '}
<span className="text-heading">{activeBranch.status}</span>
</div>
<div>
<span className="text-secondary">Steps:</span>{' '}
<span className="text-muted-foreground">Steps:</span>{' '}
<span className="text-heading">{activeBranch.step_count}</span>
</div>
<div>
<span className="text-secondary">Order:</span>{' '}
<span className="text-muted-foreground">Order:</span>{' '}
<span className="text-heading">{activeBranch.branch_order}</span>
</div>
</div>
{activeBranch.status_reason && (
<div className="mt-2 pt-2 border-t border-default">
<span className="text-secondary">Reason:</span>{' '}
<span className="text-muted-foreground">Reason:</span>{' '}
<span className="text-primary">{activeBranch.status_reason}</span>
</div>
)}
{activeBranch.context_summary && (
<div className="mt-2 pt-2 border-t border-default">
<span className="text-secondary">Tried:</span>{' '}
<span className="text-muted-foreground">Tried:</span>{' '}
<span className="text-primary">{activeBranch.context_summary.tried.join(', ')}</span>
<br />
<span className="text-secondary">Concluded:</span>{' '}
<span className="text-muted-foreground">Concluded:</span>{' '}
<span className="text-primary">{activeBranch.context_summary.concluded}</span>
</div>
)}
@@ -570,7 +570,7 @@ export default function DevBranchingPage() {
function SectionLabel({ children }: { children: React.ReactNode }) {
return (
<span className="text-[10px] font-semibold uppercase tracking-wider text-secondary mb-2 block">
<span className="text-[10px] font-semibold uppercase tracking-wider text-muted-foreground mb-2 block">
{children}
</span>
)

View File

@@ -37,7 +37,7 @@ export default function SessionQueuePage() {
<span className="text-sm font-medium text-heading">{item.problem_summary || 'Untitled session'}</span>
{item.priority === 'elevated' && <span className="text-[10px] px-1.5 py-0.5 rounded-full bg-danger-dim text-danger">Elevated</span>}
</div>
{item.engineer_notes && <p className="text-xs text-secondary">{item.engineer_notes}</p>}
{item.engineer_notes && <p className="text-xs text-muted-foreground">{item.engineer_notes}</p>}
<div className="flex items-center gap-2 mt-1 text-xs text-muted">
<Clock size={10} />
<span>{new Date(item.created_at).toLocaleString()}</span>