From a9edf4d2ab7d3dc566fdfcd0bac2b37b074a78ab Mon Sep 17 00:00:00 2001 From: Jackson Harper Date: Tue, 8 Aug 2023 13:51:11 +0800 Subject: [PATCH] Make sure we remove event listener so it isnt triggered multiple times This fixes a bug where multiple delete confirmation snackbars could show when deleting with keyboard controls. --- packages/web/pages/[username]/[slug]/index.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/web/pages/[username]/[slug]/index.tsx b/packages/web/pages/[username]/[slug]/index.tsx index a5a729bc6..0bad57608 100644 --- a/packages/web/pages/[username]/[slug]/index.tsx +++ b/packages/web/pages/[username]/[slug]/index.tsx @@ -195,6 +195,7 @@ export default function Home(): JSX.Element { return () => { document.removeEventListener('archive', archive) document.removeEventListener('mark-read', markRead) + document.removeEventListener('delete', deletePage) document.removeEventListener('openOriginalArticle', openOriginalArticle) } }, [actionHandler]) @@ -213,7 +214,6 @@ export default function Home(): JSX.Element { const deleteCurrentItem = useCallback(async () => { if (article) { const pageId = article.id - removeItemFromCache(cache, mutate, pageId) await deleteLinkMutation(pageId).then((res) => { if (res) {