Skip to content

Commit 56e327a

Browse files
committed
Remove custom renders to ensure visual pairity with Dialog
1 parent ceb4337 commit 56e327a

File tree

2 files changed

+1
-67
lines changed

2 files changed

+1
-67
lines changed

packages/react/src/ConfirmationDialog/ConfirmationDialog.module.css

Lines changed: 0 additions & 26 deletions
This file was deleted.

packages/react/src/ConfirmationDialog/ConfirmationDialog.tsx

Lines changed: 1 addition & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,9 @@
11
import type React from 'react'
22
import {useCallback} from 'react'
33
import {createRoot} from 'react-dom/client'
4-
import {FocusKeys} from '@primer/behaviors'
5-
import type {DialogProps, DialogHeaderProps, DialogButtonProps, DialogWidth, DialogHeight} from '../Dialog/Dialog'
4+
import type {DialogButtonProps, DialogWidth, DialogHeight} from '../Dialog/Dialog'
65
import {Dialog} from '../Dialog/Dialog'
7-
import {useFocusZone} from '../hooks/useFocusZone'
86
import BaseStyles from '../BaseStyles'
9-
import classes from './ConfirmationDialog.module.css'
10-
import Heading from '../Heading'
117

128
/**
139
* Props to customize the ConfirmationDialog.
@@ -81,39 +77,6 @@ export interface ConfirmationDialogProps {
8177
height?: DialogHeight
8278
}
8379

84-
const ConfirmationHeader: React.FC<React.PropsWithChildren<DialogHeaderProps>> = ({title, onClose, dialogLabelId}) => {
85-
const onCloseClick = useCallback(() => {
86-
onClose('close-button')
87-
}, [onClose])
88-
89-
return (
90-
<div className={classes.ConfirmationHeader}>
91-
<Heading id={dialogLabelId} as="h1" variant="small">
92-
{title}
93-
</Heading>
94-
<Dialog.CloseButton onClose={onCloseClick} />
95-
</div>
96-
)
97-
}
98-
99-
const ConfirmationBody: React.FC<React.PropsWithChildren<DialogProps>> = ({children}) => {
100-
return <div className={classes.ConfirmationBody}>{children}</div>
101-
}
102-
103-
const ConfirmationFooter: React.FC<React.PropsWithChildren<DialogProps>> = ({footerButtons}) => {
104-
const {containerRef: footerRef} = useFocusZone({
105-
bindKeys: FocusKeys.ArrowHorizontal | FocusKeys.Tab,
106-
focusInStrategy: 'closest',
107-
})
108-
109-
// Must have exactly 2 buttons!
110-
return (
111-
<div ref={footerRef as React.RefObject<HTMLDivElement>} className={classes.ConfirmationFooter}>
112-
<Dialog.Buttons buttons={footerButtons ?? []} />
113-
</div>
114-
)
115-
}
116-
11780
/**
11881
* A ConfirmationDialog is a special kind of dialog with more rigid behavior. It
11982
* is used to confirm a user action. ConfirmationDialogs always have exactly
@@ -168,9 +131,6 @@ export const ConfirmationDialog: React.FC<React.PropsWithChildren<ConfirmationDi
168131
width={width}
169132
height={height}
170133
className={className}
171-
renderHeader={ConfirmationHeader}
172-
renderBody={ConfirmationBody}
173-
renderFooter={ConfirmationFooter}
174134
>
175135
{children}
176136
</Dialog>

0 commit comments

Comments
 (0)