Skip to content

Commit

Permalink
feat: Add yellow value to status indicator colorOverride prop (#1682)
Browse files Browse the repository at this point in the history
  • Loading branch information
jperals authored Oct 24, 2023
1 parent a86eadc commit 3d3418e
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion pages/status-indicator/permutations.page.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ const permutations = createPermutations<InternalStatusIndicatorProps>([
},
{
type: ['pending'],
colorOverride: ['blue', 'grey', 'green', 'red'],
colorOverride: ['blue', 'grey', 'green', 'red', 'yellow'],
},
{
type: ['info'],
Expand Down
1 change: 1 addition & 0 deletions src/__tests__/__snapshots__/documenter.test.ts.snap
Original file line number Diff line number Diff line change
Expand Up @@ -12455,6 +12455,7 @@ Object {
"grey",
"green",
"red",
"yellow",
],
},
"name": "colorOverride",
Expand Down
2 changes: 1 addition & 1 deletion src/status-indicator/internal.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ export namespace StatusIndicatorProps {
// Why not enums? Explained there
// https://stackoverflow.com/questions/52393730/typescript-string-literal-union-type-from-enum
export type Type = 'error' | 'warning' | 'success' | 'info' | 'stopped' | 'pending' | 'in-progress' | 'loading';
export type Color = 'blue' | 'grey' | 'green' | 'red';
export type Color = 'blue' | 'grey' | 'green' | 'red' | 'yellow';
}

export default function StatusIndicator({
Expand Down
1 change: 1 addition & 0 deletions src/status-indicator/styles.scss
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ $_color-overrides: (
'grey': awsui.$color-text-status-inactive,
'blue': awsui.$color-text-status-info,
'green': awsui.$color-text-status-success,
'yellow': awsui.$color-text-status-warning,
);

.root {
Expand Down

0 comments on commit 3d3418e

Please sign in to comment.