Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 22 additions & 7 deletions src/components/drawer/Drawer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ export interface DrawerProps {
onClose: () => void
level?: 'primary' | 'secondary'
header?: React.ReactNode | string
disableFocusTrap?: boolean
}

const DRAWER_PORTAL_ID = 'drawer-portal'
Expand All @@ -22,6 +23,7 @@ export const DrawerPanel: React.FC<DrawerProps> = ({
onClose,
level = 'primary',
header,
disableFocusTrap = false,
}) => {
const globalShellEnabled =
useSystemSettingsStore(
Expand All @@ -47,7 +49,11 @@ export const DrawerPanel: React.FC<DrawerProps> = ({
onClick={(e) => e.stopPropagation()}
>
{isOpen && (
<DrawerContents onClose={onClose} header={header}>
<DrawerContents
onClose={onClose}
header={header}
disableFocusTrap={disableFocusTrap}
>
{children}
</DrawerContents>
)}
Expand All @@ -60,10 +66,12 @@ const DrawerContents = ({
children,
onClose,
header,
disableFocusTrap,
}: {
children: React.ReactNode
onClose: () => void
header?: React.ReactNode | string
disableFocusTrap?: boolean
}) => {
useEffect(() => {
const onKeyDown = (e: KeyboardEvent) => e.key === 'Escape' && onClose()
Expand All @@ -76,7 +84,18 @@ const DrawerContents = ({
) : (
header
)
return (

const content = (
<div className={css.drawerContent}>
<span tabIndex={0}></span>
{headerNode}
<div className={css.drawerBody}>{children}</div>
</div>
)

return disableFocusTrap ? (
content
) : (
<FocusTrap
focusTrapOptions={{
delayInitialFocus: true,
Expand All @@ -86,11 +105,7 @@ const DrawerContents = ({
},
}}
>
<div className={css.drawerContent}>
<span tabIndex={0}></span>
{headerNode}
<div className={css.drawerBody}>{children}</div>
</div>
{content}
</FocusTrap>
)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ const OptionListNewOrEdit = () => {
: i18n.t('Edit option')}
</DrawerHeader>
}
disableFocusTrap={true}
>
<EditOrNewOptionForm
onSubmitted={onSubmitted}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,7 @@ const ProgramStageListNewOrEdit = () => {
: i18n.t('Edit stage')}
</DrawerHeader>
}
disableFocusTrap={true}
>
{stageFormOpen !== undefined && (
<EditOrNewStageForm
Expand Down
Loading