fix: fix small ui issues after merge of multiple PRs

_locales/en/messages.json

@@ -41,7 +41,7 @@
   "unreadMessages": "Unread Messages",
   "debugLogExplanation": "This log will be saved to your desktop.",
   "reportIssue": "Report a Bug",
-  "markAllAsRead": "Mark All as Read",
+  "markAllAsRead": "Mark Read",
   "incomingError": "Error handling incoming message",
   "media": "Media",
   "mediaEmptyState": "No media",
@@ -79,7 +79,7 @@
   "typingAlt": "Typing animation for this conversation",
   "contactAvatarAlt": "Avatar for contact $name$",
   "downloadAttachment": "Download Attachment",
-  "replyToMessage": "Reply to message",
+  "replyToMessage": "Reply",
   "replyingToMessage": "Replying to:",
   "originalMessageNotFound": "Original message not found",
   "you": "You",
@@ -107,13 +107,10 @@
   "deleteMessagesConfirmation": "Permanently delete the messages in this conversation?",
   "hideConversation": "Hide Conversation",
   "hideNoteToSelfConfirmation": "Are you sure you want to hide your <b>Note to Self</b> conversation?",
-  "hideConversationFailed": "Failed to hide the Conversation!",
-  "hideConversationFailedPleaseTryAgain": "Unable to hide the conversation, please try again",
   "deleteConversation": "Delete Conversation",
   "deleteConversationConfirmation": "Are you sure you want to delete your conversation with <b>$name$</b>?",
   "deleteConversationFailed": "Failed to delete the Conversation!",
   "deleteConversationFailedPleaseTryAgain": "Unable to delete the conversation, please try again",
-  "hiding": "Hiding...",
   "leaving": "Leaving...",
   "deleted": "$count$ deleted",
   "messageDeletedPlaceholder": "This message has been deleted",
@@ -126,6 +123,7 @@
   "groupMembers": "Members",
   "moreInformation": "More information",
   "failed": "Failed",
+  "failedToSendMessage": "Failed to send message",
   "read": "Read",
   "resend": "Resend",
   "clear": "Clear",

stylesheets/_modules.scss

@@ -554,7 +554,8 @@
   justify-content: center;
   align-items: center;
   flex-grow: 1;
-  font-size: 28px;
+  margin-top: var(--margins-sm);
+  font-size: 16px;
 }
 
 // Module: Conversation List Item

ts/components/conversation/message/message-content/MessageContentWithStatus.tsx

@@ -132,7 +132,7 @@ export const MessageContentWithStatuses = (props: Props) => {
         onDoubleClickCapture={onDoubleClickReplyToMessage}
         dataTestId={dataTestId}
       >
-        <Flex container={true} flexDirection="column" flexShrink={0}>
+        <Flex container={true} flexDirection="column" flexShrink={0} alignItems="flex-end">
           <StyledMessageWithAuthor>
             {!isDetailView && <MessageAuthorText messageId={messageId} />}
             <MessageContent messageId={messageId} isDetailView={isDetailView} />

ts/components/conversation/message/message-content/MessageStatus.tsx

@@ -30,12 +30,11 @@ type Props = {
  *        - if the message is incoming: do not show anything (3)
  *        - if the message is outgoing: show the text for the last message, or a message sending, or in the error state. (4)
  */
-export const MessageStatus = (props: Props) => {
+export const MessageStatus = ({ isDetailView, messageId, dataTestId }: Props) => {
-  const { messageId, isDetailView, dataTestId } = props;
+  const status = useMessageStatus(messageId);
-  const status = useMessageStatus(props.messageId);
+  const selected = useMessageExpirationPropsById(messageId);
-  const selected = useMessageExpirationPropsById(props.messageId);
 
-  if (!props.messageId || !selected || isDetailView) {
+  if (!messageId || !selected || isDetailView) {
     return null;
   }
   const isIncoming = selected.direction === 'incoming';
@@ -79,15 +78,15 @@ const MessageStatusContainer = styled.div<{ isIncoming: boolean; isGroup: boolea
     props.isGroup || !props.isIncoming ? 'var(--width-avatar-group-msg-list)' : 0};
 `;
 
-const StyledStatusText = styled.div`
+const StyledStatusText = styled.div<{ textColor: string }>`
-  color: var(--text-secondary-color);
   font-size: small;
+  color: ${props => props.textColor};
 `;
 
-const TextDetails = ({ text }: { text: string }) => {
+const TextDetails = ({ text, textColor }: { text: string; textColor: string }) => {
   return (
     <>
-      <StyledStatusText>{text}</StyledStatusText>
+      <StyledStatusText textColor={textColor}>{text}</StyledStatusText>
       <SpacerXS />
     </>
   );
@@ -154,7 +153,7 @@ const MessageStatusSending = ({ dataTestId }: Omit<Props, 'isDetailView'>) => {
       isIncoming={false}
       isGroup={false}
     >
-      <TextDetails text={window.i18n('sending')} />
+      <TextDetails text={window.i18n('sending')} textColor="var(--text-secondary-color)" />
       <IconNormal rotateDuration={2} iconType="sending" />
     </MessageStatusContainer>
   );
@@ -193,7 +192,7 @@ const MessageStatusSent = ({ dataTestId, messageId }: Omit<Props, 'isDetailView'
       isIncoming={false}
       isGroup={isGroup}
     >
-      <TextDetails text={window.i18n('sent')} />
+      <TextDetails text={window.i18n('sent')} textColor="var(--text-secondary-color)" />
       <IconForExpiringMessageId messageId={messageId} iconType="circleCheck" />
     </MessageStatusContainer>
   );
@@ -221,7 +220,7 @@ const MessageStatusRead = ({
       isIncoming={isIncoming}
       isGroup={isGroup}
     >
-      <TextDetails text={window.i18n('read')} />
+      <TextDetails text={window.i18n('read')} textColor="var(--text-secondary-color)" />
       <IconForExpiringMessageId messageId={messageId} iconType="doubleCheckCircleFilled" />
     </MessageStatusContainer>
   );
@@ -243,7 +242,7 @@ const MessageStatusError = ({ dataTestId }: Omit<Props, 'isDetailView'>) => {
       isIncoming={false}
       isGroup={isGroup}
     >
-      <TextDetails text={window.i18n('failed')} />
+      <TextDetails text={window.i18n('failedToSendMessage')} textColor="var(--danger-color)" />
       <IconDanger iconType="error" />
     </MessageStatusContainer>
   );

ts/components/conversation/message/message-item/InteractionNotification.tsx

@@ -38,7 +38,7 @@ export const InteractionNotification = (props: PropsForInteractionNotification)
 
   switch (interactionType) {
     case ConversationInteractionType.Hide:
-      text = window.i18n('hideConversationFailedPleaseTryAgain');
+      // this can't happen
       break;
     case ConversationInteractionType.Leave:
       text = isCommunity

ts/components/conversation/right-panel/RightPanel.tsx

@@ -2,6 +2,7 @@ import React from 'react';
 
 import styled from 'styled-components';
 import { useRightOverlayMode } from '../../../hooks/useUI';
+import { isRtlBody } from '../../../util/i18n';
 import { Flex } from '../../basic/Flex';
 import { OverlayRightPanelSettings } from './overlay/OverlayRightPanelSettings';
 import { OverlayDisappearingMessages } from './overlay/disappearing-messages/OverlayDisappearingMessages';
@@ -113,6 +114,8 @@ const ClosableOverlay = () => {
 };
 
 export const RightPanel = () => {
+  const isRtlMode = isRtlBody();
+
   return (
     <StyledRightPanel
       container={true}
@@ -121,6 +124,7 @@ export const RightPanel = () => {
       width={'var(--right-panel-width)'}
       height={'var(--right-panel-height)'}
       className="right-panel"
+      style={{ direction: isRtlMode ? 'rtl' : 'initial' }}
     >
       <ClosableOverlay />
     </StyledRightPanel>

ts/components/conversation/right-panel/overlay/OverlayRightPanelSettings.tsx

@@ -42,7 +42,6 @@ import { AttachmentTypeWithPath } from '../../../../types/Attachment';
 import { getAbsoluteAttachmentPath } from '../../../../types/MessageAttachment';
 import { Avatar, AvatarSize } from '../../../avatar/Avatar';
 import { Flex } from '../../../basic/Flex';
-import { SessionButton, SessionButtonColor, SessionButtonType } from '../../../basic/SessionButton';
 import { SpacerMD } from '../../../basic/Text';
 import { PanelButtonGroup, PanelIconButton } from '../../../buttons';
 import { MediaItemType } from '../../../lightbox/LightboxGallery';
@@ -183,25 +182,6 @@ const HeaderItem = () => {
   );
 };
 
-const StyledLeaveButton = styled.div`
-  width: 100%;
-  .session-button {
-    margin-top: auto;
-    width: 100%;
-    min-height: calc(var(--composition-container-height) + 1px); // include border in height
-    flex-shrink: 0;
-    align-items: center;
-    border-top: 1px solid var(--border-color);
-    border-radius: 0px;
-
-    &:not(.disabled) {
-      &:hover {
-        background-color: var(--button-solid-background-hover-color);
-      }
-    }
-  }
-`;
-
 const StyledName = styled.h4`
   padding-inline: var(--margins-md);
   font-size: var(--font-size-md);
@@ -359,15 +339,14 @@ export const OverlayRightPanelSettings = () => {
 
         <MediaGallery documents={documents} media={media} />
         {isGroup && (
-          <StyledLeaveButton>
+          <PanelIconButton
-            <SessionButton
             text={leaveGroupString}
-              buttonColor={SessionButtonColor.Danger}
+            dataTestId="leave-group-button"
-              buttonType={SessionButtonType.Simple}
             disabled={isKickedFromGroup || left}
-              onClick={deleteConvoAction}
+            onClick={() => void deleteConvoAction()}
+            color={'var(--danger-color)'}
+            iconType={'delete'}
           />
-          </StyledLeaveButton>
         )}
       </PanelButtonGroup>
     </>

ts/components/conversation/right-panel/overlay/message-info/components/AttachmentInfo.tsx

@@ -10,7 +10,8 @@ type Props = {
 
 const StyledLabelContainer = styled(Flex)`
   div {
-    flex-grow: 1;
+    min-width: 50%;
+    // we want 2 items per row and that's the easiest to make it happen
   }
 `;
 
@@ -23,7 +24,7 @@ export const AttachmentInfo = (props: Props) => {
         label={`${window.i18n('fileId')}:`}
         info={attachment?.id ? String(attachment.id) : window.i18n('notApplicable')}
       />
-      <StyledLabelContainer container={true} flexDirection="row" justifyContent="space-between">
+      <StyledLabelContainer container={true} flexDirection="row" flexWrap="wrap">
         <LabelWithInfo
           label={`${window.i18n('fileType')}:`}
           info={
@@ -34,8 +35,6 @@ export const AttachmentInfo = (props: Props) => {
           label={`${window.i18n('fileSize')}:`}
           info={attachment?.fileSize ? String(attachment.fileSize) : window.i18n('notApplicable')}
         />
-      </StyledLabelContainer>
-      <StyledLabelContainer container={true} flexDirection="row" justifyContent="space-between">
         <LabelWithInfo
           label={`${window.i18n('resolution')}:`}
           info={

ts/components/leftpane/conversation-list-item/InteractionItem.tsx

@@ -58,14 +58,7 @@ export const InteractionItem = (props: InteractionItemProps) => {
 
   switch (interactionType) {
     case ConversationInteractionType.Hide:
-      errorText = window.i18n('hideConversationFailed');
+      // if it's hidden or pending hiding, we don't show it any text
-      text =
-        interactionStatus === ConversationInteractionStatus.Error
-          ? errorText
-          : interactionStatus === ConversationInteractionStatus.Start ||
-            interactionStatus === ConversationInteractionStatus.Loading
-          ? window.i18n('hiding')
-          : text;
       break;
     case ConversationInteractionType.Leave:
       errorText = isCommunity

ts/interactions/conversationInteractions.ts

@@ -294,12 +294,6 @@ async function leaveGroupOrCommunityByConvoId(
   onClickClose?: () => void
 ) {
   try {
-    await updateConversationInteractionState({
-      conversationId,
-      type: ConversationInteractionType.Leave,
-      status: ConversationInteractionStatus.Start,
-    });
-
     if (onClickClose) {
       onClickClose();
     }
@@ -308,13 +302,20 @@ async function leaveGroupOrCommunityByConvoId(
       await getConversationController().deleteCommunity(conversationId, {
         fromSyncMessage: false,
       });
-    } else {
+      return;
+    }
+    // for groups, we have a "leaving..." state that we don't need for communities.
+    // that's because communities can be left always, whereas for groups we need to send a leave message (and so have some encryption keypairs)
+    await updateConversationInteractionState({
+      conversationId,
+      type: ConversationInteractionType.Leave,
+      status: ConversationInteractionStatus.Start,
+    });
     await getConversationController().deleteClosedGroup(conversationId, {
       fromSyncMessage: false,
       sendLeaveMessage: true,
       forceDeleteLocal,
     });
-    }
     await clearConversationInteractionState({ conversationId });
   } catch (err) {
     window.log.warn(`showLeaveGroupByConvoId error: ${err}`);

ts/interactions/conversations/unsendingInteractions.ts

@@ -347,6 +347,8 @@ export async function deleteMessagesByIdForEveryone(
   const messageCount = selectedMessages.length;
   const moreThanOne = messageCount > 1;
 
+  const closeDialog = () => window.inboxStore?.dispatch(updateConfirmModal(null));
+
   window.inboxStore?.dispatch(
     updateConfirmModal({
       title: window.i18n('deleteForEveryone'),
@@ -359,8 +361,9 @@ export async function deleteMessagesByIdForEveryone(
         await doDeleteSelectedMessages({ selectedMessages, conversation, deleteForEveryone: true });
 
         // explicitly close modal for this case.
-        window.inboxStore?.dispatch(updateConfirmModal(null));
+        closeDialog();
       },
+      onClickCancel: closeDialog,
       closeAfterInput: false,
     })
   );

ts/models/message.ts

@@ -1435,7 +1435,8 @@ export class MessageModel extends Backbone.Model<MessageAttributes> {
 
           switch (interactionType) {
             case ConversationInteractionType.Hide:
-              return window.i18n('hideConversationFailed');
+              // there is no text for hiding changes
+              return '';
             case ConversationInteractionType.Leave:
               return isCommunity
                 ? window.i18n('leaveCommunityFailed')

ts/session/conversations/ConversationController.ts

@@ -519,7 +519,6 @@ async function leaveClosedGroup(groupId: string, fromSyncMessage: boolean) {
   });
 
   window?.log?.info(`We are leaving the group ${groupId}. Sending our leaving message.`);
-
   // if we do not have a keypair for that group, we can't send our leave message, so just skip the message sending part
   const wasSent = await getMessageQueue().sendToPubKeyNonDurably({
     message: ourLeavingMessage,

ts/types/LocalizerKeys.ts

@@ -202,6 +202,7 @@ export type LocalizerKeys =
   | 'failedResolveOns'
   | 'failedToAddAsModerator'
   | 'failedToRemoveFromModerator'
+  | 'failedToSendMessage'
   | 'faq'
   | 'fileId'
   | 'fileSize'
@@ -221,14 +222,11 @@ export type LocalizerKeys =
   | 'hide'
   | 'hideBanner'
   | 'hideConversation'
-  | 'hideConversationFailed'
-  | 'hideConversationFailedPleaseTryAgain'
   | 'hideMenuBarDescription'
   | 'hideMenuBarTitle'
   | 'hideNoteToSelfConfirmation'
   | 'hideRequestBanner'
   | 'hideRequestBannerDescription'
-  | 'hiding'
   | 'iAmSure'
   | 'image'
   | 'imageAttachmentAlt'