Skip to content

Commit

Permalink
fix(ui): Fixed mobile transaction edit.
Browse files Browse the repository at this point in the history
Resolves #1312
  • Loading branch information
elliotcourant committed Feb 18, 2023
1 parent e24e4cd commit d4a8298
Showing 1 changed file with 8 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,13 @@ function EditTransactionDialogMobile(props: EditTransactionDialogMobileProps): J
const { transaction } = props;

async function closeDialog() {
modal.hide()
.then(() => modal.remove());
modal.keepMounted = false;
modal.hide();
setTimeout(() => modal.remove(), 500);
// TODO The promise from `modal.hide()` never resolves, so modal.remove is never called; causing issues with
// the same modal being re-used here.
// modal.hide()
// .then(() => console.warn(modal.remove()));
}

async function validateInput(input: EditTransactionForm): FormikErrors<EditTransactionForm> {
Expand Down Expand Up @@ -100,6 +105,7 @@ function EditTransactionDialogMobile(props: EditTransactionDialogMobileProps): J

return (
<SwipeableDrawer
transitionDuration={ 250 }
style={ {
zIndex: 1100,
} }
Expand Down

0 comments on commit d4a8298

Please sign in to comment.