Extract pencil icon to UIIMage+Woo

ctarda · ctarda · commit ae3267684cf7 · 2019-03-08T05:38:16.000+01:00
diff --git a/WooCommerce/Classes/Extensions/UIImage+Woo.swift b/WooCommerce/Classes/Extensions/UIImage+Woo.swift
@@ -34,6 +34,15 @@ extension UIImage {
         return UIImage(named: "gravatar")!
     }
 
+    /// Pencil Icon
+    ///
+    static var pencilImage: UIImage {
+        let tintColor = StyleManager.wooCommerceBrandColor
+        return Gridicon.iconOfType(.pencil)
+            .imageWithTintColor(tintColor)!
+            .imageFlippedForRightToLeftLayoutDirection()
+    }
+
     /// Jetpack Logo Image
     ///
     static var jetpackLogoImage: UIImage {
diff --git a/WooCommerce/Classes/ViewRelated/Orders/Cells/SummaryTableViewCell.swift b/WooCommerce/Classes/ViewRelated/Orders/Cells/SummaryTableViewCell.swift
@@ -109,10 +109,7 @@ private extension SummaryTableViewCell {
     }
 
     func configureIcon() {
-        let pencilIcon = Gridicon.iconOfType(.pencil)
-            .imageWithTintColor(tintColor)?
-            .imageFlippedForRightToLeftLayoutDirection()
-        updateStatusButton.setImage(pencilIcon, for: .normal)
+        updateStatusButton.setImage(.pencilImage, for: .normal)
 
         updateStatusButton.addTarget(self, action: #selector(editWasTapped), for: .touchUpInside)
 

Original file line number	Diff line number	Diff line change
`@@ -109,10 +109,7 @@ private extension SummaryTableViewCell {`
`109`	`109`	`}`
`110`	`110`
`111`	`111`	`func configureIcon() {`
`112`		`- let pencilIcon = Gridicon.iconOfType(.pencil)`
`113`		`- .imageWithTintColor(tintColor)?`
`114`		`- .imageFlippedForRightToLeftLayoutDirection()`
`115`		`- updateStatusButton.setImage(pencilIcon, for: .normal)`
	`112`	`+ updateStatusButton.setImage(.pencilImage, for: .normal)`
`116`	`113`
`117`	`114`	`updateStatusButton.addTarget(self, action: #selector(editWasTapped), for: .touchUpInside)`
`118`	`115`