fix: Cannot delete message with attachments without content (#6378)

This commit is contained in:
Sivin Varghese
2023-02-03 00:33:19 +05:30
committed by GitHub
parent c9667190f6
commit 84e46adf1c
2 changed files with 21 additions and 15 deletions

View File

@@ -105,7 +105,8 @@
v-if="isBubble && !isMessageDeleted"
:is-open="showContextMenu"
:show-copy="hasText"
:show-canned-response-option="isOutgoing"
:show-delete="hasTextOrAttachment"
:show-canned-response-option="isOutgoing && hasText"
:menu-position="contextMenuPosition"
:message-content="data.content"
@toggle="handleContextMenuClick"
@@ -308,6 +309,9 @@ export default {
hasText() {
return !!this.data.content;
},
hasTextOrAttachment() {
return this.hasText || this.data.attachments.length > 0;
},
tooltipForSender() {
const name = this.senderNameForAvatar;
const { message_type: messageType } = this.data;

View File

@@ -24,18 +24,18 @@
:class="`dropdown-pane--${menuPosition}`"
>
<woot-dropdown-menu>
<woot-dropdown-item v-if="showDelete">
<woot-button
variant="clear"
color-scheme="alert"
size="small"
icon="delete"
@click="handleDelete"
>
{{ $t('CONVERSATION.CONTEXT_MENU.DELETE') }}
</woot-button>
</woot-dropdown-item>
<woot-dropdown-item v-if="showCopy">
<woot-dropdown-item>
<woot-button
variant="clear"
color-scheme="alert"
size="small"
icon="delete"
@click="handleDelete"
>
{{ $t('CONVERSATION.CONTEXT_MENU.DELETE') }}
</woot-button>
</woot-dropdown-item>
<woot-button
variant="clear"
size="small"
@@ -46,10 +46,8 @@
{{ $t('CONVERSATION.CONTEXT_MENU.COPY') }}
</woot-button>
</woot-dropdown-item>
<woot-dropdown-item>
<woot-dropdown-item v-if="showCannedResponseOption">
<woot-button
v-if="showCannedResponseOption"
variant="clear"
size="small"
icon="comment-add"
@@ -94,6 +92,10 @@ export default {
type: Boolean,
default: false,
},
showDelete: {
type: Boolean,
default: false,
},
menuPosition: {
type: String,
default: 'left',