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
This commit is contained in:
Lim Chee Aun 2022-12-16 00:53:04 +08:00
parent 859500a292
commit 03d2e2aa72

View file

@ -271,6 +271,7 @@ function Compose({
const isSelf = replyToStatus?.account.id === currentAccount; const isSelf = replyToStatus?.account.id === currentAccount;
const hasOnlyAcct = const hasOnlyAcct =
replyToStatus && value.trim() === `@${replyToStatus.account.acct}`; replyToStatus && value.trim() === `@${replyToStatus.account.acct}`;
// TODO: check for mentions, or maybe just generic "@username<space>", including multiple mentions like "@username1<space>@username2<space>"
if (!isSelf && hasOnlyAcct) { if (!isSelf && hasOnlyAcct) {
console.log('canClose', { isSelf, hasOnlyAcct }); console.log('canClose', { isSelf, hasOnlyAcct });
return true; return true;