Fix Connecting spinner in dark mode, maintain draft attachment order

stylesheets/_modules.scss

@@ -4443,10 +4443,22 @@ button.module-image__border-overlay:focus {
 .module-spinner__circle--small {
   -webkit-mask: url('../images/spinner-track-24.svg') no-repeat center;
   -webkit-mask-size: 100%;
+
+  // For specificity
+  @include dark-theme {
+    -webkit-mask: url('../images/spinner-track-24.svg') no-repeat center;
+    -webkit-mask-size: 100%;
+  }
 }
 .module-spinner__arc--small {
   -webkit-mask: url('../images/spinner-24.svg') no-repeat center;
   -webkit-mask-size: 100%;
+
+  // For specificity
+  @include dark-theme {
+    -webkit-mask: url('../images/spinner-24.svg') no-repeat center;
+    -webkit-mask-size: 100%;
+  }
 }
 
 .module-spinner__circle--incoming {

stylesheets/components/LeftPaneDialog.scss

@@ -33,6 +33,11 @@
   &__spinner {
     &__arc {
       background-color: $color-black;
+
+      // Needed for specificity
+      @include dark-theme {
+        background-color: $color-black;
+      }
     }
 
     &__circle {

ts/views/conversation_view.ts

@@ -1913,12 +1913,24 @@ export class ConversationView extends window.Backbone.View<ConversationModel> {
     const onDisk = await this.writeDraftAttachment(attachment);
 
     // Remove any pending attachments that were transcoding
-    const draftAttachments = (this.model.get('draftAttachments') || []).filter(
+    const draftAttachments = this.model.get('draftAttachments') || [];
-      draftAttachment => draftAttachment.path !== attachment.path
+    const index = draftAttachments.findIndex(
+      draftAttachment => draftAttachment.path === attachment.path
     );
-    this.model.set({
+    if (index < 0) {
-      draftAttachments: [onDisk, ...draftAttachments],
+      window.log.warn(
-    });
+        `addAttachment: Failed to find pending attachment with path ${attachment.path}`
+      );
+      this.model.set({
+        draftAttachments: [...draftAttachments, onDisk],
+      });
+    } else {
+      const toUpdate = [...draftAttachments];
+      toUpdate.splice(index, 1, onDisk);
+      this.model.set({
+        draftAttachments: toUpdate,
+      });
+    }
     this.updateAttachmentsView();
 
     await this.saveModel();