From 84b20496ed6ba8cfc107b5e592d117cc74906a9d Mon Sep 17 00:00:00 2001 From: automated-signal <37887102+automated-signal@users.noreply.github.com> Date: Fri, 19 Apr 2024 11:40:20 -0500 Subject: [PATCH] Close note preview modal when clearing note Co-authored-by: Jamie Kyle <113370520+jamiebuilds-signal@users.noreply.github.com> --- ts/state/smart/EditNicknameAndNoteModal.tsx | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/ts/state/smart/EditNicknameAndNoteModal.tsx b/ts/state/smart/EditNicknameAndNoteModal.tsx index 46605b58d3..15e961df70 100644 --- a/ts/state/smart/EditNicknameAndNoteModal.tsx +++ b/ts/state/smart/EditNicknameAndNoteModal.tsx @@ -25,14 +25,19 @@ export const SmartEditNicknameAndNoteModal = memo( 'EditNicknameAndNoteModal requires conversation' ); - const { toggleEditNicknameAndNoteModal } = useGlobalModalActions(); + const { toggleEditNicknameAndNoteModal, toggleNotePreviewModal } = + useGlobalModalActions(); const { updateNicknameAndNote } = useConversationsActions(); const handleSave = useCallback( (nicknameAndNote: NicknameAndNote) => { + // Ensure we don't re-open the note preview modal if there's no note. + if (nicknameAndNote.note == null) { + toggleNotePreviewModal(null); + } updateNicknameAndNote(conversationId, nicknameAndNote); }, - [conversationId, updateNicknameAndNote] + [conversationId, updateNicknameAndNote, toggleNotePreviewModal] ); const handleClose = useCallback(() => {