Skip to content

Commit

Permalink
[Security Solution] [Cases] EUI Refresh: Change cases in progress sta…
Browse files Browse the repository at this point in the history
…tus color (#206240)

## Summary

This PR resolves #205740
Change the `In Progress` badge to the hollow style to make it less
prominent.

<img width="1066" alt="Screenshot 2025-01-10 at 10 45 46"
src="https://github.com/user-attachments/assets/c66ac32b-75e4-42c2-908d-690c41e2aeba"
/>
<img width="1054" alt="Screenshot 2025-01-10 at 10 46 07"
src="https://github.com/user-attachments/assets/c82f2910-9be5-4de1-a6ab-14e47019ccab"
/>


### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Elastic Machine <[email protected]>
  • Loading branch information
agusruidiazgd and elasticmachine authored Jan 24, 2025
1 parent c22b982 commit 3805b46
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,12 @@ import { CaseStatuses } from './types';

export const getStatusConfiguration = () => ({
[CaseStatuses.open]: {
color: 'primary',
color: 'success',
label: i18n.OPEN,
icon: 'folderOpen' as const,
},
[CaseStatuses['in-progress']]: {
color: 'warning',
color: 'primary',
label: i18n.IN_PROGRESS,
icon: 'folderExclamation' as const,
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ describe('StatusPopoverButton', () => {
.find(`[data-test-subj="case-status-badge-popover-button-open"]`)
.first()
.prop('color')
).toBe('primary');
).toBe('success');
});

it('shows the correct color when status is in-progress', async () => {
Expand All @@ -82,7 +82,7 @@ describe('StatusPopoverButton', () => {
.find(`[data-test-subj="case-status-badge-popover-button-in-progress"]`)
.first()
.prop('color')
).toBe('warning');
).toBe('primary');
});

it('shows the correct color when status is closed', async () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -110,14 +110,14 @@ const CasesByStatusComponent: React.FC = () => {
const barColors = useMemo(
() => ({
empty: euiTheme.colors.vis.euiColorVis8,
open: euiTheme.colors.primary,
'in-progress': euiTheme.colors.warning,
open: euiTheme.colors.success,
'in-progress': euiTheme.colors.primary,
closed: euiTheme.colors.borderBaseSubdued,
}),
[
euiTheme.colors.vis.euiColorVis8,
euiTheme.colors.primary,
euiTheme.colors.warning,
euiTheme.colors.success,
euiTheme.colors.borderBaseSubdued,
]
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ interface Props {

const statuses = {
[CaseStatuses.open]: {
color: 'primary',
color: 'success',
label: i18n.STATUS_OPEN,
},
[CaseStatuses['in-progress']]: {
color: 'warning',
color: 'primary',
label: i18n.STATUS_IN_PROGRESS,
},
[CaseStatuses.closed]: {
Expand Down

0 comments on commit 3805b46

Please sign in to comment.