Skip to content

Commit

Permalink
Removes red/green colorscheme for a11y reasons (#2090)
Browse files Browse the repository at this point in the history
  • Loading branch information
alexcreasy committed Nov 10, 2023
1 parent 6858ebd commit 4e321d5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import {
ModelServingMetricsContext,
} from '~/pages/modelServing/screens/metrics/ModelServingMetricsContext';
import { ContextResourceData, PrometheusQueryRangeResultValue } from '~/types';
import { SUCCESS_FAIL_CHART_THEME } from '~/pages/modelServing/screens/metrics/const';
import { per100 } from './utils';

const ModelGraphs: React.FC = () => {
Expand All @@ -32,8 +31,8 @@ const ModelGraphs: React.FC = () => {
translatePoint: per100,
},
]}
color="blue"
title="Http requests (x100)"
theme={SUCCESS_FAIL_CHART_THEME}
isStack
/>
</StackItem>
Expand Down
1 change: 1 addition & 0 deletions frontend/src/pages/modelServing/screens/metrics/const.ts
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,7 @@ const themeProps = {
chart: { colorScale },
group: { colorScale },
legend: { colorScale },
stack: { colorScale },
};

export const SUCCESS_FAIL_CHART_THEME = mergeTheme(ChartThemeColor.default, themeProps);

0 comments on commit 4e321d5

Please sign in to comment.