Skip to content

Commit

Permalink
Change design of delete source modal (#84)
Browse files Browse the repository at this point in the history
Signed-off-by: Montse Ortega <[email protected]>
  • Loading branch information
ammont82 authored Jan 30, 2025
1 parent 7989fea commit 2a9d50c
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
4 changes: 2 additions & 2 deletions apps/demo/src/components/ConfirmationModal.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ export const ConfirmationModal: React.FC<

return (
<Modal
width="24rem"
width="44rem"
isOpen={isOpen}
variant={variant}
aria-describedby="modal-title-icon-description"
Expand All @@ -68,7 +68,7 @@ export const ConfirmationModal: React.FC<
isDisabled={isDisabled}
onClick={onConfirm}
>
Confirm
Delete
</Button>
{onCancel && (
<Button key="cancel" variant="link" onClick={onCancel}>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,17 +64,18 @@ export const RemoveSourceAction: React.FC<RemoveSourceAction.Props> = (
</Tooltip>
{onConfirm && shouldShowConfirmationModal && (
<ConfirmationModal
title="Remove discovery source?"
title="Delete Environment"
titleIconVariant="warning"
primaryButtonVariant="danger"
isOpen={shouldShowConfirmationModal}
isDisabled={isDisabled}
onCancel={dismissConfirmationModal}
onConfirm={handleConfirm}
onClose={dismissConfirmationModal}
>
<TextContent>
<Text id="confirmation-modal-description">
The discovery information will be lost.
Are you sure you want to delete this environment?
<br/>To use it again, create a new discovery image and redeploy it.
</Text>
</TextContent>
</ConfirmationModal>
Expand Down

0 comments on commit 2a9d50c

Please sign in to comment.