From 368595178d73868470121c8ca8cf375679e8a553 Mon Sep 17 00:00:00 2001 From: Osamaali313 Date: Sat, 18 Jul 2026 23:36:43 +0300 Subject: [PATCH] Fix missing "Poor" label on the average response-time gauge MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `RadialAvgResponse` maps the response-time palette to a qualitative word: const palette = getResponseTimeColor(avg); const msg: Record = { success: "Excellent", warning: "Average", danger: "Poor", }; ... {msg[palette]} But `getResponseTimeColor` only ever returns `"success" | "warning" | "error"` (MonitorUtils.ts) — never `"danger"`. So for the slow case `palette` is `"error"`, `msg["error"]` is `undefined`, and no label renders. The same component uses `theme.palette[palette].main`, which works precisely because `palette` is the real MUI key `"error"`; every other palette helper in MonitorUtils.ts also uses `"error"`, and `"danger"` appears nowhere else in the client. Any monitor whose average response time is >= 300 ms shows an empty label exactly where the "Poor" signal matters most. Use the `error` key. --- client/src/Components/monitors/charts/RadialAvgResponse.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/client/src/Components/monitors/charts/RadialAvgResponse.tsx b/client/src/Components/monitors/charts/RadialAvgResponse.tsx index 9a88fb42c..dccb18984 100644 --- a/client/src/Components/monitors/charts/RadialAvgResponse.tsx +++ b/client/src/Components/monitors/charts/RadialAvgResponse.tsx @@ -20,7 +20,7 @@ export const RadialAvgResponse = ({ avg, max }: { avg: number; max: number }) => const msg: Record = { success: "Excellent", warning: "Average", - danger: "Poor", + error: "Poor", }; return (