Completed message selection

_locales/en/messages.json

@@ -997,6 +997,10 @@
   "delete": {
     "message": "Delete"
   },
+  "forwardMessage": {
+    "message": "Forward",
+    "description": "Text of Forward Message button"
+  },
   "deletePublicWarning": {
     "message":
       "Are you sure? Clicking 'delete' will permanently remove this message for everyone in this channel."

js/models/conversations.js

@@ -590,6 +590,7 @@
         onCopyPublicKey: () => this.copyPublicKey(),
         onDeleteContact: () => this.deleteContact(),
         onDeleteMessages: () => this.deleteMessages(),
+        onCloseOverlay: () => this.resetMessageSelection(),
       };
 
       this.updateAsyncPropsCache();

js/views/conversation_view.js

@@ -152,11 +152,6 @@
         'show-message-detail',
         this.showMessageDetail
       );
-      this.listenTo(
-        this.model,
-        'message-selection-changed',
-        this.onMessageSelectionChanged
-      );
       this.listenTo(this.model.messageCollection, 'navigate-to', url => {
         window.location = url;
       });
@@ -241,6 +236,8 @@
           onSetDisappearingMessages: seconds =>
             this.setDisappearingMessages(seconds),
           onDeleteMessages: () => this.destroyMessages(),
+          onDeleteSelectedMessages: () => this.deleteSelectedMessages(),
+          onCloseOverlay: () => this.model.resetMessageSelection(),
           onDeleteContact: () => this.model.deleteContact(),
           onResetSession: () => this.endSession(),
 
@@ -1814,18 +1811,6 @@
       }
     },
 
-    onMessageSelectionChanged() {
-      const selectionSize = this.model.selectedMessages.size;
-
-      if (selectionSize > 0) {
-        $('.compose').hide();
-      } else {
-        $('.compose').show();
-      }
-
-      this.bulkEditView.update(selectionSize);
-    },
-
     resetMessageSelection() {
       this.model.resetMessageSelection();
     },

ts/components/conversation/ConversationHeader.tsx

@@ -62,6 +62,9 @@ interface Props {
   onDeleteContact: () => void;
   onResetSession: () => void;
 
+  onCloseOverlay: () => void;
+  onDeleteSelectedMessages: () => void;
+
   onArchive: () => void;
   onMoveToInbox: () => void;
 
@@ -246,7 +249,6 @@ export class ConversationHeader extends React.Component<Props> {
         <SessionIconButton
           iconType={SessionIconType.Ellipses}
           iconSize={SessionIconSize.Large}
-          onClick={this.showMenuBound}
         />
       </ContextMenuTrigger>
     );
@@ -302,7 +304,7 @@ export class ConversationHeader extends React.Component<Props> {
   }
 
   public renderSelectionOverlay() {
-    const { onDeleteMessages } = this.props;
+    const { onDeleteSelectedMessages, onCloseOverlay, i18n } = this.props;
 
     return (
       <div className="message-selection-overlay">
@@ -310,6 +312,7 @@ export class ConversationHeader extends React.Component<Props> {
           <SessionIconButton
             iconType={SessionIconType.Exit}
             iconSize={SessionIconSize.Medium}
+            onClick={onCloseOverlay}
           />
         </div>
 
@@ -317,15 +320,14 @@ export class ConversationHeader extends React.Component<Props> {
           <SessionButton
             buttonType={SessionButtonType.Default}
             buttonColor={SessionButtonColor.Primary}
-            text={'Forward'}
+            text={i18n('forwardMessage')}
-            onClick={onDeleteMessages}
           />
 
           <SessionButton
             buttonType={SessionButtonType.Default}
             buttonColor={SessionButtonColor.Danger}
-            text={'Delete'}
+            text={i18n('delete')}
-            onClick={onDeleteMessages}
+            onClick={onDeleteSelectedMessages}
           />
         </div>
       </div>

ts/components/session/icon/SessionIcon.tsx

@@ -9,7 +9,6 @@ export interface Props {
   iconColor: string;
   iconPadded: boolean;
   iconRotation: number;
-  onClick: any;
 }
 
 export class SessionIcon extends React.PureComponent<Props> {
@@ -18,7 +17,6 @@ export class SessionIcon extends React.PureComponent<Props> {
     iconColor: '',
     iconRotation: 0,
     iconPadded: false,
-    onClick: () => null,
   };
 
   constructor(props: any) {

ts/components/session/icon/SessionIconButton.tsx

@@ -3,8 +3,18 @@ import classNames from 'classnames';
 
 import { Props, SessionIcon } from '../icon';
 
-export class SessionIconButton extends React.PureComponent<Props> {
+interface SProps extends Props {
-  public static defaultProps = SessionIcon.defaultProps;
+  onClick: any;
+}
+
+export class SessionIconButton extends React.PureComponent<SProps> {
+  public static readonly extendedDefaults = {
+    onClick: () => null,
+  };
+  public static readonlydefaultProps = {
+    ...SessionIcon.defaultProps,
+    ...SessionIconButton.extendedDefaults,
+  };
 
   constructor(props: any) {
     super(props);
@@ -18,7 +28,6 @@ export class SessionIconButton extends React.PureComponent<Props> {
       iconColor,
       iconRotation,
       iconPadded,
-      onClick,
     } = this.props;
 
     return (
@@ -38,7 +47,6 @@ export class SessionIconButton extends React.PureComponent<Props> {
           iconSize={iconSize}
           iconColor={iconColor}
           iconRotation={iconRotation}
-          onClick={onClick}
         />
       </div>
     );