From be2e9473b3f2b6a7a53cb1539e68135a48267213 Mon Sep 17 00:00:00 2001 From: Kevin G Christiano Date: Mon, 3 Feb 2020 09:00:03 -0500 Subject: [PATCH] Removing func as the static dictionary can carry the weight. --- MVMCoreUI/Categories/UIColor+Extension.swift | 7 +------ MVMCoreUI/CustomPrimitives/Color.swift | 4 ++-- 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/MVMCoreUI/Categories/UIColor+Extension.swift b/MVMCoreUI/Categories/UIColor+Extension.swift index aec1ce17..55a10c03 100644 --- a/MVMCoreUI/Categories/UIColor+Extension.swift +++ b/MVMCoreUI/Categories/UIColor+Extension.swift @@ -6,11 +6,11 @@ // Copyright © 2019 Verizon Wireless. All rights reserved. // - import UIKit public typealias ColorHexTuple = (uiColor: UIColor, hex: String) + extension UIColor { /// Dictionary to access brand approved colors by name. @@ -200,9 +200,4 @@ extension UIColor { return nil } - - public class func getColorAndHexFromName(_ name: String) -> ColorHexTuple? { - - return names[name] - } } diff --git a/MVMCoreUI/CustomPrimitives/Color.swift b/MVMCoreUI/CustomPrimitives/Color.swift index d72e1af9..a34fbd45 100644 --- a/MVMCoreUI/CustomPrimitives/Color.swift +++ b/MVMCoreUI/CustomPrimitives/Color.swift @@ -56,7 +56,7 @@ public final class Color: Codable { } init?(name: String) { - guard let colorTuple = UIColor.getColorAndHexFromName(name) else { return nil } + guard let colorTuple = UIColor.names[name] else { return nil } self.uiColor = colorTuple.uiColor self.hex = colorTuple.hex determineRGBA() @@ -73,7 +73,7 @@ public final class Color: Codable { if colorString.hasPrefix("#") { hex = colorString.replacingOccurrences(of: "#", with: "") } else { - guard let hex = UIColor.getColorAndHexFromName(colorString)?.hex else { throw ColorError.badName(reason: "Check the spelling of your color.") } + guard let hex = UIColor.names[colorString]?.hex else { throw ColorError.badName(reason: "Check the spelling of your color.") } self.hex = hex.replacingOccurrences(of: "#", with: "") name = colorString }