[MM-51489]: Fix issue with compact message mode (#22711)

Automatic Merge
Этот коммит содержится в:
Ashish Dhama
2023-04-06 00:34:28 +05:30
коммит произвёл GitHub
родитель 54af9a7c7d
Коммит 0af69b3411
5 изменённых файлов: 18 добавлений и 9 удалений

Просмотреть файл

@@ -254,16 +254,16 @@ const PostComponent = (props: Props): JSX.Element => {
'current--user': props.currentUserId === post.user_id && !isSystemMessage, 'current--user': props.currentUserId === post.user_id && !isSystemMessage,
'post--system': isSystemMessage || isMeMessage, 'post--system': isSystemMessage || isMeMessage,
'post--root': props.hasReplies && !(post.root_id && post.root_id.length > 0), 'post--root': props.hasReplies && !(post.root_id && post.root_id.length > 0),
'post--comment': post.root_id && post.root_id.length > 0 && !props.isCollapsedThreadsEnabled, 'post--comment': (post.root_id && post.root_id.length > 0 && !props.isCollapsedThreadsEnabled) || (props.location === Locations.RHS_COMMENT),
'post--compact': props.compactDisplay, 'post--compact': props.compactDisplay,
'post--hovered': hovered, 'post--hovered': hovered,
'same--user': props.isConsecutivePost && !props.compactDisplay, 'same--user': props.isConsecutivePost && (!props.compactDisplay || props.location === Locations.RHS_COMMENT),
'cursor--pointer': alt && !props.channelIsArchived, 'cursor--pointer': alt && !props.channelIsArchived,
'post--hide-controls': post.failed || post.state === Posts.POST_DELETED, 'post--hide-controls': post.failed || post.state === Posts.POST_DELETED,
'post--comment same--root': fromAutoResponder, 'post--comment same--root': fromAutoResponder,
'post--pinned-or-flagged': (post.is_pinned || props.isFlagged) && props.location === Locations.CENTER, 'post--pinned-or-flagged': (post.is_pinned || props.isFlagged) && props.location === Locations.CENTER,
'mention-comment': props.isCommentMention, 'mention-comment': props.isCommentMention,
'post--thread': props.location === Locations.RHS_COMMENT || Locations.RHS_ROOT, 'post--thread': props.location === Locations.RHS_COMMENT || props.location === Locations.RHS_ROOT,
}); });
}; };
@@ -378,7 +378,8 @@ const PostComponent = (props: Props): JSX.Element => {
let profilePic; let profilePic;
const hideProfilePicture = hasSameRoot(props) && (!post.root_id && !props.hasReplies) && !PostUtils.isFromBot(post); const hideProfilePicture = hasSameRoot(props) && (!post.root_id && !props.hasReplies) && !PostUtils.isFromBot(post);
if (!hideProfilePicture) { const hideProfileCase = !(props.location === Locations.RHS_COMMENT && props.compactDisplay && props.isConsecutivePost);
if (!hideProfilePicture && hideProfileCase) {
profilePic = ( profilePic = (
<PostProfilePicture <PostProfilePicture
compactDisplay={props.compactDisplay} compactDisplay={props.compactDisplay}
@@ -528,7 +529,7 @@ const PostComponent = (props: Props): JSX.Element => {
isSystemMessage={isSystemMessage} isSystemMessage={isSystemMessage}
/> />
<div className='col d-flex align-items-center'> <div className='col d-flex align-items-center'>
{ {((!hideProfilePicture && props.location === Locations.CENTER) || hover || props.location !== Locations.CENTER) &&
<PostTime <PostTime
isPermalink={!(Posts.POST_DELETED === post.state || isPostPendingOrFailed(post))} isPermalink={!(Posts.POST_DELETED === post.state || isPostPendingOrFailed(post))}
eventTime={post.create_at} eventTime={post.create_at}

Просмотреть файл

@@ -19,6 +19,7 @@ import CommentIcon from 'components/common/comment_icon';
import {Emoji} from '@mattermost/types/emojis'; import {Emoji} from '@mattermost/types/emojis';
import {Post} from '@mattermost/types/posts'; import {Post} from '@mattermost/types/posts';
import classnames from 'classnames';
type Props = { type Props = {
post: Post; post: Post;
@@ -235,7 +236,7 @@ const PostOptions = (props: Props): JSX.Element => {
<div <div
ref={dotMenuRef} ref={dotMenuRef}
data-testid={`post-menu-${props.post.id}`} data-testid={`post-menu-${props.post.id}`}
className='col post-menu' className={classnames('col post-menu', {'post-menu--position': !hoverLocal && showCommentIcon})}
> >
{!collapsedThreadsEnabled && !showRecentlyUsedReactions && dotMenu} {!collapsedThreadsEnabled && !showRecentlyUsedReactions && dotMenu}
{showRecentReactions} {showRecentReactions}

Просмотреть файл

@@ -79,6 +79,8 @@
} }
&.post--compact { &.post--compact {
padding-left: 1em;
.post__body { .post__body {
padding-left: 0; padding-left: 0;
} }
@@ -102,7 +104,7 @@
&.same--user { &.same--user {
.post__header { .post__header {
.col__name { .col__name {
display: inline-block; display: flex;
} }
} }
} }

Просмотреть файл

@@ -2395,8 +2395,7 @@
.post.current--user { .post.current--user {
&.other--root, &.other--root,
&.post--root, &.post--root, {
&.post--comment {
.post__header-set-custom-status { .post__header-set-custom-status {
display: revert; display: revert;
padding: 0 6px; padding: 0 6px;

Просмотреть файл

@@ -292,6 +292,8 @@
.post__permalink { .post__permalink {
position: absolute; position: absolute;
top: 1px; top: 1px;
left: -76px;
width: 60px;
text-align: right; text-align: right;
} }
@@ -355,6 +357,10 @@
.post__header { .post__header {
.post-menu { .post-menu {
top: -34px; top: -34px;
&.post-menu--position {
top: -12px;
}
} }
} }
} }