refactor(ChevronIcon): Do not add padding

Infomaniak · Sep 5, 2023 · 130f646 · 130f646
1 parent 4c43849
commit 130f646
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 36 deletions.
diff --git a/Mail/Components/ChevronIcon.swift b/Mail/Components/ChevronIcon.swift
@@ -38,14 +38,13 @@ struct ChevronIcon: View {
     }
 
     let style: Style
+    var color = MailResourcesAsset.textSecondaryColor
 
     var body: some View {
         MailResourcesAsset.arrowDown.swiftUIImage
             .resizable()
             .frame(width: 12, height: 12)
-            .foregroundColor(MailResourcesAsset.textSecondaryColor)
-            .padding(.vertical, 2)
-            .padding(.horizontal, 1.5)
+            .foregroundColor(color)
             .rotationEffect(style.rotationAngle)
     }
 }

diff --git a/Mail/Views/Menu Drawer/MailboxManagement/MailboxesManagementButtonView.swift b/Mail/Views/Menu Drawer/MailboxManagement/MailboxesManagementButtonView.swift
@@ -67,10 +67,7 @@ struct MailboxesManagementButtonView: View {
                 } else {
                     switch style {
                     case .blockedPassword:
-                        MailResourcesAsset.arrowRight.swiftUIImage
-                            .resizable()
-                            .frame(width: 12, height: 12)
-                            .foregroundColor(MailResourcesAsset.textPrimaryColor.swiftUIColor)
+                        ChevronIcon(style: .right, color: MailResourcesAsset.textPrimaryColor)
                     case .menuDrawer:
                         if let detailNumber {
                             Text(detailNumber < 100 ? "\(detailNumber)" : "99+")

diff --git a/Mail/Views/Search/SearchFilterFolderCell.swift b/Mail/Views/Search/SearchFilterFolderCell.swift
@@ -77,9 +77,7 @@ struct SearchFilterFolderCell: View {
                 }
                 Text(selectedFolderName)
                     .font(MailTextStyle.bodyMedium.font)
-                MailResourcesAsset.arrowDown.swiftUIImage
-                    .resizable()
-                    .frame(width: 12, height: 12)
+                ChevronIcon(style: .down)
             }
         }
         .filterCellStyle(isSelected: isSelected)

diff --git a/MailResources/Assets.xcassets/arrow-right.imageset/Contents.json b/MailResources/Assets.xcassets/arrow-right.imageset/Contents.json
diff --git a/MailResources/Assets.xcassets/arrow-right.imageset/chevron-right.svg b/MailResources/Assets.xcassets/arrow-right.imageset/chevron-right.svg