docs: Explain IKIcon sizes

Infomaniak · Nov 29, 2023 · f767325 · f767325
1 parent 78b0095
commit f767325
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 17 deletions.
diff --git a/Mail/Components/Buttons/IKButton/IKButtonLabel.swift b/Mail/Components/Buttons/IKButton/IKButtonLabel.swift
@@ -27,8 +27,6 @@ extension LabelStyle where Self == IKLabelStyle {
 }
 
 struct IKLabelStyle: LabelStyle {
-    private let size = IKIcon.Size.regular
-
     func makeBody(configuration: Configuration) -> some View {
         HStack(spacing: UIPadding.small) {
             configuration.icon

diff --git a/Mail/Components/IKIcon.swift b/Mail/Components/IKIcon.swift
@@ -21,19 +21,13 @@ import MailResources
 import SwiftUI
 
 struct IKIcon: View {
-    enum Size {
-        case small, regular, large
-
-        var heightAndWidth: CGFloat {
-            switch self {
-            case .small:
-                return 12
-            case .regular:
-                return 16
-            case .large:
-                return 24
-            }
-        }
+    enum Size: CGFloat {
+        /// 12pt icon
+        case small = 12
+        /// 16pt icon
+        case regular = 16
+        /// 24pt icon
+        case large = 24
     }
 
     let icon: Image
@@ -52,10 +46,14 @@ struct IKIcon: View {
         icon
             .resizable()
             .scaledToFit()
-            .frame(width: size.heightAndWidth, height: size.heightAndWidth)
+            .frame(width: size.rawValue, height: size.rawValue)
     }
 }
 
 #Preview {
-    IKIcon(MailResourcesAsset.folder, size: .large)
+    HStack(spacing: UIPadding.regular) {
+        IKIcon(MailResourcesAsset.pencilPlain, size: .small)
+        IKIcon(MailResourcesAsset.pencilPlain, size: .regular)
+        IKIcon(MailResourcesAsset.pencilPlain, size: .large)
+    }
 }