Merge pull request #395 from GetStream/small-fixes

Small fixes

Author: szuperaz

projects/stream-chat-angular/src/lib/attachment-list/attachment-list.component.html

@@ -189,12 +189,16 @@
           "
         ></video>
       </div>
-      <div
+      <a
         *ngIf="isFile(attachment)"
         class="
           str-chat__message-attachment-file--item
           str-chat-angular__message-attachment-file-single
         "
+        style="cursor: pointer; text-decoration: none"
+        data-testclass="file-link"
+        href="{{ attachment.asset_url }}"
+        target="_blank"
       >
         <stream-icon-placeholder
           *ngIf="themeVersion === '1'"
@@ -214,7 +218,8 @@
             >
               {{ attachment.title }}
             </div>
-            <a
+            <!-- Temporary disabled, will be fixed with this: https://github.com/GetStream/stream-chat-angular/issues/393  -->
+            <!-- <a
               class="str-chat__message-attachment-file--item-download"
               data-testclass="file-link"
               download
@@ -225,7 +230,7 @@
                 class="str-chat__message-attachment-download-icon"
                 icon="download"
               ></stream-icon-placeholder>
-            </a>
+            </a> -->
           </div>
           <span
             class="str-chat__message-attachment-file--item-size"
@@ -234,7 +239,7 @@
             >{{ getFileSize(attachment) }}</span
           >
         </div>
-      </div>
+      </a>
       <div
         *ngIf="
           isCard(attachment) &&

projects/stream-chat-angular/src/lib/attachment-list/attachment-list.component.spec.ts

@@ -424,7 +424,6 @@ describe('AttachmentListComponent', () => {
       const link = queryFileLinks()[0];
       const titleElement = queryFileNames()[0];
 
-      expect(link.hasAttribute('download')).toBeTrue();
       expect(link.href).toContain(asset_url);
       expect(titleElement.textContent).toContain(title);
     });
@@ -467,7 +466,6 @@ describe('AttachmentListComponent', () => {
       fixture.detectChanges();
       const link = queryFileLinks()[0];
 
-      expect(link.hasAttribute('download')).toBeTrue();
       expect(link.href).toContain('unsafe:' + asset_url);
     });
 

projects/stream-chat-angular/src/lib/message-actions-box/message-actions-box.component.html

@@ -27,7 +27,7 @@
   <button
     class="str-chat__message-actions-list-item-button"
     [attr.data-testid]="actionName + '-action'"
-    (click)="actionHandler()"
+    (click)="actionHandler(message, isMine)"
   >
     <li class="str-chat__message-actions-list-item">
       {{ getActionLabel(actionLabelOrTranslationKey) | translate }}

projects/stream-chat-angular/src/lib/message-actions-box/message-actions-box.component.spec.ts

@@ -225,7 +225,7 @@ describe('MessageActionsBoxComponent', () => {
 
     expect(channelService.pinMessage).toHaveBeenCalledWith(component.message);
 
-    component.message!.pinned = true;
+    component.message = { ...component.message!, pinned: true };
     action?.click();
     fixture.detectChanges();
 

projects/stream-chat-angular/src/lib/message-actions-box/message-actions-box.component.ts

@@ -164,14 +164,6 @@ export class MessageActionsBoxComponent implements OnChanges, OnDestroy {
 
   ngOnChanges(changes: SimpleChanges): void {
     if (changes.isMine || changes.enabledActions || changes.message) {
-      this.messageActionItems.forEach(
-        (i) =>
-          (i.actionHandler = i.actionHandler.bind(
-            this,
-            this.message!,
-            this.isMine
-          ))
-      );
       this.visibleMessageActionItems = this.messageActionItems.filter((item) =>
         item.isVisible(this.enabledActions, this.isMine, this.message!)
       );

projects/stream-chat-angular/src/lib/message-list/message-list.component.html

@@ -5,12 +5,13 @@
   (scroll)="scrolled()"
 >
   <div class="str-chat__reverse-infinite-scroll str-chat__message-list-scroll">
-    <ul class="str-chat__ul">
+    <ul class="str-chat__ul" style="-webkit-transform: translate3d(0, 0, 0)">
       <li
         #parentMessageElement
         *ngIf="mode === 'thread'"
         data-testid="parent-message"
         class="str-chat__parent-message-li"
+        style="-webkit-transform: translate3d(0, 0, 0)"
       >
         <ng-container
           *ngTemplateOutlet="