Skip to content

Commit

Permalink
runfix: address read indicator design review (WPB-5369) (#16629)
Browse files Browse the repository at this point in the history
* runfix: assign max-width to all assets

* remove read indicator from message header
  • Loading branch information
V-Gira authored Jan 25, 2024
1 parent ae1bc2d commit c201095
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ import React, {useMemo, useState, useEffect} from 'react';

import {QualifiedId} from '@wireapp/api-client/lib/user';

import {ReadIndicator} from 'Components/MessagesList/Message/ReadIndicator';
import {Conversation} from 'src/script/entity/Conversation';
import {CompositeMessage} from 'src/script/entity/message/CompositeMessage';
import {ContentMessage} from 'src/script/entity/message/ContentMessage';
Expand Down Expand Up @@ -192,13 +191,6 @@ export const ContentMessageComponent: React.FC<ContentMessageProps> = ({
{timeAgo}
</MessageTime>
</span>

<ReadIndicator
message={message}
is1to1Conversation={conversation.is1to1()}
isLastDeliveredMessage={isLastDeliveredMessage}
showIconOnly
/>
</MessageHeader>
)}

Expand Down
1 change: 1 addition & 0 deletions src/style/components/asset/common/common.less
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
position: relative;
display: flex;
overflow: hidden;
max-width: var(--conversation-message-asset-width);
min-height: 64px;

border-radius: 12px;
Expand Down
4 changes: 0 additions & 4 deletions src/style/components/asset/link-preview-asset.less
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,8 @@

.link-preview-asset {
.asset-container-style;
}

.link-preview-asset {
display: flex;
width: 100%;
max-width: var(--conversation-message-asset-width);
cursor: pointer;

&.ephemeral-asset-expired {
Expand Down
2 changes: 1 addition & 1 deletion src/style/components/asset/video-asset.less
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,9 @@

// Video asset component
.video-asset {
.asset-container-style;
position: relative;
display: block;
max-width: var(--conversation-message-asset-width);
flex: 1 1 auto;

&__container {
Expand Down

0 comments on commit c201095

Please sign in to comment.