From 03d2e2aa72755ee35f7a9d449f61c390c8ff1a47 Mon Sep 17 00:00:00 2001 From: Lim Chee Aun Date: Fri, 16 Dec 2022 00:53:04 +0800 Subject: [PATCH] Add TODO for now for handling multiple mentions If it's just all mentions with no textual content = can close(?) But if there're a lot of mentions, won't it be dangerous to close? User can still edit whatever in between like rearranging the mentions etc --- src/components/compose.jsx | 1 + 1 file changed, 1 insertion(+) diff --git a/src/components/compose.jsx b/src/components/compose.jsx index 2b329814..469ce277 100644 --- a/src/components/compose.jsx +++ b/src/components/compose.jsx @@ -271,6 +271,7 @@ function Compose({ const isSelf = replyToStatus?.account.id === currentAccount; const hasOnlyAcct = replyToStatus && value.trim() === `@${replyToStatus.account.acct}`; + // TODO: check for mentions, or maybe just generic "@username", including multiple mentions like "@username1@username2" if (!isSelf && hasOnlyAcct) { console.log('canClose', { isSelf, hasOnlyAcct }); return true;