Skip to content

Commit

Permalink
fix: onHide error thrown on ESC press when <UpdateVersionModal /> is …
Browse files Browse the repository at this point in the history
…open
  • Loading branch information
howard-e authored Jul 17, 2024
1 parent 6b4aedd commit d118b35
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions client/components/common/BasicModal/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,8 @@ const BasicModal = ({
id={`focus-trapped-${id}`}
centered={centered}
animation={animation}
onHide={useOnHide ? handleHide || handleClose : null}
onExit={!useOnHide ? handleHide || handleClose : null}
onHide={useOnHide ? handleHide || handleClose : () => {}}
onExit={!useOnHide ? handleHide || handleClose : () => {}}
/* Disabled due to buggy implementation which jumps the page */
autoFocus={false}
aria-labelledby={`title-${id}`}
Expand Down

0 comments on commit d118b35

Please sign in to comment.