From 08d725bfb38a71bfd8107b410a3c2bc053e0c917 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andr=C3=A9=20Padez?= <andre.padez@gmail.com>
Date: Sun, 16 Aug 2026 02:47:53 +0000
Subject: [PATCH] the background job button wears the amber a running job wears
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Not white. `toneText.warning` — hsl(43 96% 56%), the same amber a running task
chip shows two rows up, so the control and the thing it produces read as one
feature rather than two.

Taken from `toneText` rather than typed as `text-warning`, so it cannot drift
from the chips it is matching. The name is misleading in this one place: nothing
here is a warning, it is just where that colour lives in the palette.

Dialog title icon follows, same reason as the glyph did.

tsgo clean, frontend builds.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 .../src/apps/Chat/components/BackgroundJobDialog.tsx     | 3 ++-
 .../src/apps/Chat/components/ModelSelector.tsx           | 9 +++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/workspaces/officerdev/src/apps/Chat/components/BackgroundJobDialog.tsx b/src/workspaces/officerdev/src/apps/Chat/components/BackgroundJobDialog.tsx
index b1d1e5cf..1b145c15 100644
--- a/src/workspaces/officerdev/src/apps/Chat/components/BackgroundJobDialog.tsx
+++ b/src/workspaces/officerdev/src/apps/Chat/components/BackgroundJobDialog.tsx
@@ -1,5 +1,6 @@
 import { useEffect, useState } from 'react';
 import { CirclePlay } from 'lucide-react';
+import { toneText } from '@/components/Data';
 import { Button } from '@/components/ui/button';
 import {
   Dialog,
@@ -87,7 +88,7 @@ export const BackgroundJobDialog = ({ open, onOpenChange, cwd, onSubmit }: Backg
       <DialogContent className="sm:max-w-lg">
         <DialogHeader>
           <DialogTitle className="flex items-center gap-2">
-            <CirclePlay className="h-4 w-4" />
+            <CirclePlay className={`h-4 w-4 ${toneText.warning}`} />
             Background job
           </DialogTitle>
           <DialogDescription>
diff --git a/src/workspaces/officerdev/src/apps/Chat/components/ModelSelector.tsx b/src/workspaces/officerdev/src/apps/Chat/components/ModelSelector.tsx
index 72636411..aec4429f 100644
--- a/src/workspaces/officerdev/src/apps/Chat/components/ModelSelector.tsx
+++ b/src/workspaces/officerdev/src/apps/Chat/components/ModelSelector.tsx
@@ -1,5 +1,6 @@
 import { useEffect } from 'react';
 import { CirclePlay } from 'lucide-react';
+import { toneText } from '@/components/Data';
 import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from '@/components/ui/select';
 import type { ModelOption } from 'state/useModels';
 import { getProviderDisplayName } from 'state/useModels';
@@ -132,13 +133,17 @@ export function ModelSelector({
           already set, and this is the one control on this row that DOES something. The earlier glyph was
           a radio tower, on the reasoning that a play button implies one-shot — but most of what gets
           started here is a long-running process, and a watcher is still something you press start on.
-          Recognisable beats precise at 18px. */}
+          Recognisable beats precise at 18px.
+
+          `warning` is not a warning here: it is the amber a RUNNING task chip wears two rows up, so the
+          button and the thing it produces are the same colour. Taken from `toneText` rather than written
+          as `text-warning`, so it cannot drift from the chips it is meant to match. */}
       {onBackgroundJob && (
         <button
           type="button"
           onClick={onBackgroundJob}
           title="Start a background job or watcher"
-          className="shrink-0 cursor-pointer rounded-md p-1.5 text-foreground transition-colors hover:bg-foreground/10"
+          className={`shrink-0 cursor-pointer rounded-md p-1.5 transition-colors hover:bg-foreground/10 ${toneText.warning}`}
         >
           <CirclePlay className="h-[18px] w-[18px]" />
         </button>