Merge pull request #899 from Infomaniak/update-file-icons

feat: Update attachments icons

Mail/Components/AttachmentView.swift

@@ -0,0 +1,71 @@
+/*
+ Infomaniak Mail - iOS App
+ Copyright (C) 2022 Infomaniak Network SA
+
+ This program is free software: you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation, either version 3 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+import MailCore
+import MailResources
+import SwiftUI
+
+struct AttachmentView<Content: View>: View {
+    let attachment: Attachment
+    let subtitle: String
+    @ViewBuilder let accessory: () -> Content?
+
+    init(attachment: Attachment, subtitle: String, accessory: @escaping () -> Content? = { EmptyView() }) {
+        self.attachment = attachment
+        self.subtitle = subtitle
+        self.accessory = accessory
+    }
+
+    var body: some View {
+        VStack(spacing: 0) {
+            HStack {
+                attachment.icon.swiftUIImage
+                    .resizable()
+                    .frame(width: 24, height: 24)
+
+                HStack(spacing: 8) {
+                    VStack(alignment: .leading, spacing: 0) {
+                        Text(attachment.name)
+                            .textStyle(.bodySmall)
+                            .lineLimit(1)
+                            .truncationMode(.middle)
+
+                        Text(subtitle)
+                            .textStyle(.labelSecondary)
+                    }
+
+                    accessory()
+                }
+            }
+            .padding(.horizontal, 8)
+            .padding(.vertical, 4)
+        }
+        .background(
+            RoundedRectangle(cornerRadius: 6)
+                .stroke(MailResourcesAsset.elementsColor.swiftUIColor, lineWidth: 1)
+        )
+        .cornerRadius(6)
+        .frame(maxWidth: 200)
+    }
+}
+
+struct AttachmentView_Previews: PreviewProvider {
+    static var previews: some View {
+        AttachmentView(attachment: PreviewHelper.sampleAttachment, subtitle: "24ko")
+    }
+}

Mail/Views/New Message/Attachments/AttachmentUploadCell.swift

@@ -21,60 +21,39 @@ import MailResources
 import SwiftUI
 
 struct AttachmentUploadCell: View {
-    let attachment: Attachment
     @ObservedObject var uploadTask: AttachmentUploadTask
+
+    let attachment: Attachment
     let attachmentRemoved: ((Attachment) -> Void)?
 
     var body: some View {
-        VStack(spacing: 0) {
-            HStack {
-                attachment.icon.swiftUIImage
-
-                VStack(alignment: .leading, spacing: 0) {
-                    Text(attachment.name)
-                        .textStyle(.bodySmall)
-                        .lineLimit(1)
-                        .truncationMode(.middle)
-                    if let error = uploadTask.error {
-                        Text(error.localizedDescription)
-                            .textStyle(.labelSecondary)
-                    } else {
-                        Text(attachment.size, format: .defaultByteCount)
-                            .textStyle(.labelSecondary)
-                            .opacity(attachment.size == 0 ? 0 : 1)
-                    }
+        AttachmentView(
+            attachment: attachment,
+            subtitle: uploadTask.error != nil ? uploadTask.error!.localizedDescription : attachment.size
+                .formatted(.defaultByteCount)
+        ) {
+            Button {
+                if let attachmentRemoved {
+                    attachmentRemoved(attachment)
                 }
-
-                Button {
-                    if let attachmentRemoved {
-                        attachmentRemoved(attachment)
-                    }
-                } label: {
-                    MailResourcesAsset.closeSmall.swiftUIImage
-                        .resizable()
-                        .foregroundColor(MailResourcesAsset.textSecondaryColor)
-                        .frame(width: 16, height: 16)
-                }
-                .buttonStyle(.borderless)
-                .padding(.leading, 8)
+            } label: {
+                MailResourcesAsset.closeSmall.swiftUIImage
+                    .resizable()
+                    .foregroundColor(MailResourcesAsset.textSecondaryColor)
+                    .frame(width: 12, height: 12)
             }
-            .padding(6)
+            .buttonStyle(.borderless)
+        }
+        .overlay(alignment: .bottom) {
             IndeterminateProgressView(indeterminate: uploadTask.progress == 0, progress: uploadTask.progress)
                 .opacity(uploadTask.progress == 1 ? 0 : 1)
         }
-        .background(
-            RoundedRectangle(cornerRadius: 6)
-                .stroke(MailResourcesAsset.elementsColor.swiftUIColor, lineWidth: 1)
-        )
-        .cornerRadius(6)
-        .frame(maxWidth: 200)
-        .padding(.top, 16)
     }
 }
 
 struct AttachmentUploadCell_Previews: PreviewProvider {
     static var previews: some View {
-        AttachmentUploadCell(attachment: PreviewHelper.sampleAttachment, uploadTask: AttachmentUploadTask()) { _ in
+        AttachmentUploadCell(uploadTask: AttachmentUploadTask(), attachment: PreviewHelper.sampleAttachment) { _ in
             /* Preview */
         }
     }

Mail/Views/New Message/Attachments/AttachmentsHeaderView.swift

@@ -30,17 +30,18 @@ struct AttachmentsHeaderView: View {
                 ScrollView(.horizontal, showsIndicators: false) {
                     HStack(spacing: 8) {
                         ForEach(attachmentsManager.attachments) { attachment in
-                            AttachmentUploadCell(attachment: attachment,
-                                                 uploadTask: attachmentsManager
-                                                     .attachmentUploadTaskOrFinishedTask(for: attachment
-                                                         .uuid)) { attachmentRemoved in
+                            AttachmentUploadCell(
+                                uploadTask: attachmentsManager.attachmentUploadTaskOrFinishedTask(for: attachment.uuid),
+                                attachment: attachment
+                            ) { attachmentRemoved in
                                 attachmentsManager.removeAttachment(attachmentRemoved)
                             }
                         }
                     }
                     .padding(.vertical, 1)
                     .padding(.horizontal, 16)
                 }
+                .padding(.top, 16)
             }
         }
         .customAlert(item: $attachmentsManager.globalError) { error in

Mail/Views/Thread/AttachmentCell.swift

@@ -24,28 +24,7 @@ struct AttachmentCell: View {
     let attachment: Attachment
 
     var body: some View {
-        VStack(spacing: 0) {
-            HStack {
-                attachment.icon.swiftUIImage
-
-                VStack(alignment: .leading, spacing: 0) {
-                    Text(attachment.name)
-                        .textStyle(.bodySmall)
-                        .lineLimit(1)
-                        .truncationMode(.middle)
-                    Text(attachment.size, format: .defaultByteCount)
-                        .textStyle(.labelSecondary)
-                        .opacity(attachment.size == 0 ? 0 : 1)
-                }
-            }
-            .padding(4)
-        }
-        .background(
-            RoundedRectangle(cornerRadius: 6)
-                .stroke(MailResourcesAsset.elementsColor.swiftUIColor, lineWidth: 1)
-        )
-        .cornerRadius(6)
-        .frame(maxWidth: 200)
+        AttachmentView(attachment: attachment, subtitle: attachment.size.formatted(.defaultByteCount))
     }
 }
 

MailCore/Models/Attachment.swift

@@ -53,24 +53,35 @@ public class Attachment: /* Hashable, */ EmbeddedObject, Codable, Identifiable {
 
     public var icon: MailResourcesImages {
         guard let uti else { return MailResourcesAsset.unknownFile }
-        if uti.conforms(to: .audio) {
-            return MailResourcesAsset.audioFile
-        } else if uti.conforms(to: .archive) {
-            return MailResourcesAsset.zipFile
+
+        if uti.conforms(to: .pdf) {
+            return MailResourcesAsset.pdfFile
+        } else if uti.conforms(to: .calendarEvent) || uti.conforms(to: .ics) {
+            return MailResourcesAsset.icsFile
+        } else if uti.conforms(to: .vCard) {
+            return MailResourcesAsset.vcardFile
         } else if uti.conforms(to: .image) {
-            return MailResourcesAsset.imageFileLandscape
-        } else if uti.conforms(to: .pdf) {
-            return MailResourcesAsset.officeFileAdobe
-        } else if uti.conforms(to: .plainText) {
-            return MailResourcesAsset.commonFileText
-        } else if uti.conforms(to: .presentation) {
-            return MailResourcesAsset.officeFileGraph
-        } else if uti.conforms(to: .spreadsheet) {
-            return MailResourcesAsset.officeFileSheet
+            return MailResourcesAsset.imageFile
+        } else if uti.conforms(to: .audio) {
+            return MailResourcesAsset.audioFile
         } else if uti.conforms(to: .movie) {
-            return MailResourcesAsset.videoFilePlay
+            return MailResourcesAsset.videoFile
+        } else if uti.conforms(to: .spreadsheet) {
+            return MailResourcesAsset.gridFile
+        } else if uti.conforms(to: .presentation) {
+            return MailResourcesAsset.pointFile
+        } else if uti.conforms(to: .sourceCode) || uti.conforms(to: .html) || uti.conforms(to: .json) || uti.conforms(to: .xml) {
+            return MailResourcesAsset.codeFile
+        } else if uti.conforms(to: .text) || uti.conforms(to: .pages) || uti.conforms(to: .onlyOffice)
+            || uti.conforms(to: .wordDoc) || uti.conforms(to: .wordDocm) || uti.conforms(to: .wordDocx) {
+            return MailResourcesAsset.docFile
+        } else if uti.conforms(to: .archive) {
+            return MailResourcesAsset.archiveFile
+        } else if uti.conforms(to: .font) {
+            return MailResourcesAsset.fontFile
+        } else {
+            return MailResourcesAsset.unknownFile
         }
-        return MailResourcesAsset.unknownFile
     }
 
     private enum CodingKeys: String, CodingKey {

MailCore/Utils/UTType+Extension.swift

@@ -0,0 +1,30 @@
+/*
+ Infomaniak Mail - iOS App
+ Copyright (C) 2022 Infomaniak Network SA
+
+ This program is free software: you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation, either version 3 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License
+ along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+import Foundation
+import UniformTypeIdentifiers
+
+public extension UTType {
+    static let pages = UTType("com.apple.iwork.pages.sffpages")!
+    static let wordDoc = UTType("com.microsoft.word.doc")!
+    static let wordDocm = UTType(mimeType: "application/vnd.ms-word")!
+    static let wordDocx = UTType("org.openxmlformats.wordprocessingml.document")!
+    static let onlyOffice = UTType("org.oasis-open.opendocument.text")!
+
+    static let ics = UTType(mimeType: "application/ics")!
+}

MailResources/Assets.xcassets/Files/archive-file.imageset/Contents.json

@@ -0,0 +1,25 @@
+{
+  "images" : [
+    {
+      "filename" : "archive-file-light.svg",
+      "idiom" : "universal"
+    },
+    {
+      "appearances" : [
+        {
+          "appearance" : "luminosity",
+          "value" : "dark"
+        }
+      ],
+      "filename" : "archive-file-dark.svg",
+      "idiom" : "universal"
+    }
+  ],
+  "info" : {
+    "author" : "xcode",
+    "version" : 1
+  },
+  "properties" : {
+    "preserves-vector-representation" : true
+  }
+}

MailResources/Assets.xcassets/Files/audio-file.imageset/Contents.json

@@ -1,7 +1,17 @@
 {
   "images" : [
     {
-      "filename" : "audio-file.svg",
+      "filename" : "audio-file-light.svg",
+      "idiom" : "universal"
+    },
+    {
+      "appearances" : [
+        {
+          "appearance" : "luminosity",
+          "value" : "dark"
+        }
+      ],
+      "filename" : "audio-file-dark.svg",
       "idiom" : "universal"
     }
   ],