Skip to content

Commit fad1b36

Browse files
authored
fix: message content padding for giphy or imgur attachments (#3412)
1 parent c39e723 commit fad1b36

1 file changed

Lines changed: 13 additions & 2 deletions

File tree

package/src/components/Message/MessageSimple/MessageContent.tsx

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ import {
2727
} from '../../../contexts/translationContext/TranslationContext';
2828

2929
import { components, primitives } from '../../../theme';
30+
import { FileTypes } from '../../../types/types';
3031
import { checkMessageEquality, checkQuotedMessageEquality } from '../../../utils/utils';
3132
import { Poll } from '../../Poll/Poll';
3233
import { useMessageData } from '../hooks/useMessageData';
@@ -570,17 +571,27 @@ export const MessageContent = (props: MessageContentProps) => {
570571
const messageHasSingleFile =
571572
messageContentOrder.length === 1 && messageContentOrder[0] === 'files' && isSingleFile;
572573
const messageHasOnlyText = messageContentOrder.length === 1 && messageContentOrder[0] === 'text';
574+
const messageHasGiphyOrImgur =
575+
otherAttachments.filter(
576+
(file) => file.type === FileTypes.Giphy || file.type === FileTypes.Imgur,
577+
).length > 0;
573578

574579
const hidePaddingTop =
575-
messageHasPoll || messageHasSingleImage || messageHasSingleFile || messageHasOnlyText;
580+
messageHasPoll ||
581+
messageHasSingleImage ||
582+
messageHasSingleFile ||
583+
messageHasOnlyText ||
584+
messageHasGiphyOrImgur;
576585

577-
const hidePaddingHorizontal = messageHasPoll || messageHasSingleImage || messageHasSingleFile;
586+
const hidePaddingHorizontal =
587+
messageHasPoll || messageHasSingleImage || messageHasSingleFile || messageHasGiphyOrImgur;
578588

579589
const hidePaddingBottom =
580590
messageHasPoll ||
581591
messageHasSingleImage ||
582592
messageHasSingleFile ||
583593
messageHasOnlyText ||
594+
messageHasGiphyOrImgur ||
584595
(messageContentOrder.length > 1 &&
585596
messageContentOrder[messageContentOrder.length - 1] === 'text');
586597

0 commit comments

Comments
 (0)