From daf6e7ea012df1e5c7ac1bd3aad6447ad01452b1 Mon Sep 17 00:00:00 2001 From: "Chintakrinda, Arun Kumar (Arun)" Date: Tue, 4 Aug 2020 16:53:40 +0530 Subject: [PATCH 1/5] Updated Opacity as per design team comments --- MVMCoreUI/Utility/MVMCoreUICommonViewsUtility.m | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MVMCoreUI/Utility/MVMCoreUICommonViewsUtility.m b/MVMCoreUI/Utility/MVMCoreUICommonViewsUtility.m index 4fa079ad..84ad5f5d 100644 --- a/MVMCoreUI/Utility/MVMCoreUICommonViewsUtility.m +++ b/MVMCoreUI/Utility/MVMCoreUICommonViewsUtility.m @@ -272,7 +272,7 @@ static const CGFloat VertialShadowOffset = 6; view.layer.shadowColor = [UIColor blackColor].CGColor; view.layer.shadowOffset = CGSizeMake(0.0f, VertialShadowOffset); - view.layer.shadowOpacity = 0.7f; + view.layer.shadowOpacity = 0.25f; view.layer.shadowPath = shadowPath.CGPath; } From 5d89ae873d91cafeead4e57f4558ff35353f3310 Mon Sep 17 00:00:00 2001 From: "Chintakrinda, Arun Kumar (Arun)" Date: Wed, 5 Aug 2020 15:45:13 +0530 Subject: [PATCH 2/5] Corner radius added to Image and bgImageContainer --- MVMCoreUI/Atomic/Atoms/Views/ImageViewModel.swift | 2 ++ MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift | 5 +++++ .../Molecules/OtherContainers/BGImageMolecule.swift | 12 ++++++++++-- .../OtherContainers/BGImageMoleculeModel.swift | 4 ++++ 4 files changed, 21 insertions(+), 2 deletions(-) diff --git a/MVMCoreUI/Atomic/Atoms/Views/ImageViewModel.swift b/MVMCoreUI/Atomic/Atoms/Views/ImageViewModel.swift index 637b074d..57a82052 100644 --- a/MVMCoreUI/Atomic/Atoms/Views/ImageViewModel.swift +++ b/MVMCoreUI/Atomic/Atoms/Views/ImageViewModel.swift @@ -20,6 +20,7 @@ import Foundation public var height: CGFloat? public var contentMode: UIView.ContentMode? public var localBundle: Bundle? + public var cornerRadius: CGFloat? public init(image: String, imageFormat: String? = nil, width: CGFloat? = nil, height: CGFloat? = nil) { self.image = image @@ -38,5 +39,6 @@ import Foundation case width case height case contentMode + case cornerRadius } } diff --git a/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift b/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift index baa63ee8..4357537c 100644 --- a/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift +++ b/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift @@ -247,6 +247,11 @@ import UIKit if let contentMode = imageModel.contentMode { imageView.contentMode = contentMode } + + if let cornerRadius = imageModel.cornerRadius { + imageView.clipsToBounds = true + imageView.layer.cornerRadius = cornerRadius + } } // MARK: - load functions diff --git a/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMolecule.swift b/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMolecule.swift index 1c39975e..e7d354ea 100644 --- a/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMolecule.swift +++ b/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMolecule.swift @@ -19,7 +19,15 @@ open class BGImageMolecule: MoleculeContainer { } open override func set(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?, _ additionalData: [AnyHashable: Any]?) { - image.setOptional(with: (model as? BGImageMoleculeModel)?.image, delegateObject, additionalData) - super.set(with: model, delegateObject, additionalData) + guard let castModel = model as? BGImageMoleculeModel else { + super.set(with: model, delegateObject, additionalData) + return + } + image.setOptional(with: castModel.image, delegateObject, additionalData) + super.set(with: castModel, delegateObject, additionalData) + if let cornerRadius = castModel.cornerRadius { + layer.cornerRadius = cornerRadius + clipsToBounds = true + } } } diff --git a/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMoleculeModel.swift b/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMoleculeModel.swift index 83b5931d..f0200455 100644 --- a/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMoleculeModel.swift +++ b/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMoleculeModel.swift @@ -13,6 +13,7 @@ open class BGImageMoleculeModel: MoleculeContainerModel { return "bgImageContainer" } public var image: ImageViewModel + public var cornerRadius: CGFloat? open override func setDefaults() { if useHorizontalMargins == nil { @@ -31,6 +32,7 @@ open class BGImageMoleculeModel: MoleculeContainerModel { private enum CodingKeys: String, CodingKey { case image + case cornerRadius } public init(_ image: ImageViewModel, molecule: MoleculeModelProtocol) { @@ -41,6 +43,7 @@ open class BGImageMoleculeModel: MoleculeContainerModel { required public init(from decoder: Decoder) throws { let typeContainer = try decoder.container(keyedBy: CodingKeys.self) image = try typeContainer.decode(ImageViewModel.self, forKey:.image) + cornerRadius = try typeContainer.decodeIfPresent(CGFloat.self, forKey: .cornerRadius) try super.init(from: decoder) } @@ -48,5 +51,6 @@ open class BGImageMoleculeModel: MoleculeContainerModel { try super.encode(to: encoder) var container = encoder.container(keyedBy: CodingKeys.self) try container.encode(image, forKey: .image) + try container.encodeIfPresent(cornerRadius, forKey: .cornerRadius) } } From d388f5be8448f3649b3f8633b481bc5fb01e7ba3 Mon Sep 17 00:00:00 2001 From: "Chintakrinda, Arun Kumar (Arun)" Date: Thu, 6 Aug 2020 20:21:14 +0530 Subject: [PATCH 3/5] Added bgColor implementation in image --- MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift | 4 ++++ .../Molecules/OtherContainers/BGImageMolecule.swift | 12 ++---------- .../OtherContainers/BGImageMoleculeModel.swift | 4 ---- 3 files changed, 6 insertions(+), 14 deletions(-) diff --git a/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift b/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift index 4357537c..2359f5b3 100644 --- a/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift +++ b/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift @@ -252,6 +252,10 @@ import UIKit imageView.clipsToBounds = true imageView.layer.cornerRadius = cornerRadius } + + if let backgroundColor = imageModel.backgroundColor?.uiColor { + imageView.backgroundColor = backgroundColor + } } // MARK: - load functions diff --git a/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMolecule.swift b/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMolecule.swift index e7d354ea..1c39975e 100644 --- a/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMolecule.swift +++ b/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMolecule.swift @@ -19,15 +19,7 @@ open class BGImageMolecule: MoleculeContainer { } open override func set(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?, _ additionalData: [AnyHashable: Any]?) { - guard let castModel = model as? BGImageMoleculeModel else { - super.set(with: model, delegateObject, additionalData) - return - } - image.setOptional(with: castModel.image, delegateObject, additionalData) - super.set(with: castModel, delegateObject, additionalData) - if let cornerRadius = castModel.cornerRadius { - layer.cornerRadius = cornerRadius - clipsToBounds = true - } + image.setOptional(with: (model as? BGImageMoleculeModel)?.image, delegateObject, additionalData) + super.set(with: model, delegateObject, additionalData) } } diff --git a/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMoleculeModel.swift b/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMoleculeModel.swift index f0200455..83b5931d 100644 --- a/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMoleculeModel.swift +++ b/MVMCoreUI/Atomic/Molecules/OtherContainers/BGImageMoleculeModel.swift @@ -13,7 +13,6 @@ open class BGImageMoleculeModel: MoleculeContainerModel { return "bgImageContainer" } public var image: ImageViewModel - public var cornerRadius: CGFloat? open override func setDefaults() { if useHorizontalMargins == nil { @@ -32,7 +31,6 @@ open class BGImageMoleculeModel: MoleculeContainerModel { private enum CodingKeys: String, CodingKey { case image - case cornerRadius } public init(_ image: ImageViewModel, molecule: MoleculeModelProtocol) { @@ -43,7 +41,6 @@ open class BGImageMoleculeModel: MoleculeContainerModel { required public init(from decoder: Decoder) throws { let typeContainer = try decoder.container(keyedBy: CodingKeys.self) image = try typeContainer.decode(ImageViewModel.self, forKey:.image) - cornerRadius = try typeContainer.decodeIfPresent(CGFloat.self, forKey: .cornerRadius) try super.init(from: decoder) } @@ -51,6 +48,5 @@ open class BGImageMoleculeModel: MoleculeContainerModel { try super.encode(to: encoder) var container = encoder.container(keyedBy: CodingKeys.self) try container.encode(image, forKey: .image) - try container.encodeIfPresent(cornerRadius, forKey: .cornerRadius) } } From 1d29966d1b6da532613c7b23c0113badcdee69ad Mon Sep 17 00:00:00 2001 From: "Chintakrinda, Arun Kumar (Arun)" Date: Thu, 6 Aug 2020 22:35:29 +0530 Subject: [PATCH 4/5] calling super and adding corner radius to container rather than imageView in it --- MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift b/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift index 2359f5b3..bd4d1387 100644 --- a/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift +++ b/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift @@ -249,13 +249,11 @@ import UIKit } if let cornerRadius = imageModel.cornerRadius { - imageView.clipsToBounds = true - imageView.layer.cornerRadius = cornerRadius + clipsToBounds = true + layer.cornerRadius = cornerRadius } - if let backgroundColor = imageModel.backgroundColor?.uiColor { - imageView.backgroundColor = backgroundColor - } + super.set(with: model, delegateObject, additionalData) } // MARK: - load functions From 0c1c7b31f0c7d4d46801f4415523809e38713b3d Mon Sep 17 00:00:00 2001 From: "Chintakrinda, Arun Kumar (Arun)" Date: Fri, 7 Aug 2020 16:32:56 +0530 Subject: [PATCH 5/5] Moved super call above --- MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift b/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift index bd4d1387..532056cb 100644 --- a/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift +++ b/MVMCoreUI/Atomic/Atoms/Views/LoadImageView.swift @@ -223,6 +223,8 @@ import UIKit public override func set(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?, _ additionalData: [AnyHashable: Any]?) { self.delegateObject = delegateObject + super.set(with: model, delegateObject, additionalData) + guard let imageModel = model as? ImageViewModel else { return } if let accessibilityString = imageModel.accessibilityText { imageView.accessibilityLabel = accessibilityString @@ -252,8 +254,6 @@ import UIKit clipsToBounds = true layer.cornerRadius = cornerRadius } - - super.set(with: model, delegateObject, additionalData) } // MARK: - load functions