diff --git a/MVMCoreUI/Atomic/Atoms/Buttons/PillButton.swift b/MVMCoreUI/Atomic/Atoms/Buttons/PillButton.swift index a9989b3b..7334ccf5 100644 --- a/MVMCoreUI/Atomic/Atoms/Buttons/PillButton.swift +++ b/MVMCoreUI/Atomic/Atoms/Buttons/PillButton.swift @@ -36,7 +36,7 @@ open class PillButton: VDS.Button, MVMCoreUIViewConstrainingProtocol, MFButtonPr text = viewModel.title isEnabled = viewModel.enabled size = viewModel.size - use = viewModel.style ?? .primary + use = viewModel.style surface = viewModel.inverted ? .dark : .light if let accessibilityText = viewModel.accessibilityText { accessibilityLabel = accessibilityText diff --git a/MVMCoreUI/Atomic/Atoms/Views/CheckboxLabel.swift b/MVMCoreUI/Atomic/Atoms/Views/CheckboxLabel.swift index 965656a6..2ce1254b 100644 --- a/MVMCoreUI/Atomic/Atoms/Views/CheckboxLabel.swift +++ b/MVMCoreUI/Atomic/Atoms/Views/CheckboxLabel.swift @@ -64,9 +64,8 @@ bottomLabelConstraint.isActive = true alignCheckbox(.center) - isAccessibilityElement = true - accessibilityHint = checkbox.accessibilityHint - accessibilityTraits = checkbox.accessibilityTraits + isAccessibilityElement = false + accessibilityElements = [checkbox, label] observation = observe(\.checkbox.isSelected, options: [.new]) { [weak self] _, _ in self?.updateAccessibilityLabel() } @@ -139,6 +138,8 @@ open func updateAccessibilityLabel() { checkbox.updateAccessibilityLabel() - accessibilityLabel = [checkbox.accessibilityLabel, label.text].compactMap { $0 }.joined(separator: ",") + if let text = label.text { + checkbox.accessibilityLabel?.append(", \(text)") + } } } diff --git a/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotification.swift b/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotification.swift index 4c677029..cef93935 100644 --- a/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotification.swift +++ b/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotification.swift @@ -10,6 +10,7 @@ import Foundation import Combine import Dispatch import MVMCore +import VDSTokens @objcMembers open class CollapsableNotification: View { //-------------------------------------------------- @@ -50,7 +51,7 @@ import MVMCore open override func reset() { super.reset() verticalStack.reset() - backgroundColor = .mvmGreen() + backgroundColor = bottomView.backgroundColor } open func subscribeForNotifications() { @@ -98,6 +99,8 @@ import MVMCore guard let model = model as? CollapsableNotificationModel else { return } topView.set(with: model, delegateObject, additionalData) bottomView.set(with: model, delegateObject, additionalData) + topView.label.textColorConfiguration = bottomView.titleLabel.textColorConfiguration + topView.label.surface = bottomView.surface // Update top view default noop to expand. if let topAction = model.topAction, @@ -110,6 +113,7 @@ import MVMCore } } initialState() + backgroundColor = bottomView.backgroundColor } open func performBlockOperation(with block: @escaping (MVMCoreBlockOperation) -> Void) { @@ -214,7 +218,7 @@ import MVMCore extension CollapsableNotification: StatusBarUI { public func getStatusBarUI() -> (color: UIColor, style: UIStatusBarStyle) { - let color = backgroundColor ?? UIColor.mvmGreen + let color = backgroundColor ?? VDSColor.feedbackInformationBackgroundOnlight var greyScale: CGFloat = 0 topView.label.textColor.getWhite(&greyScale, alpha: nil) return (color, greyScale > 0.5 ? .lightContent : .default) @@ -226,7 +230,7 @@ extension CollapsableNotification: AccessibilityProtocol { if !topView.isHidden { return topView } else { - return bottomView.headline + return bottomView.titleLabel } } } diff --git a/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotificationModel.swift b/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotificationModel.swift index 3d366347..4c87a4e4 100644 --- a/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotificationModel.swift +++ b/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotificationModel.swift @@ -26,21 +26,13 @@ open class CollapsableNotificationModel: NotificationMoleculeModel { self.collapseTime = collapseTime } super.init(with: headline, style: style, backgroundColor: backgroundColor, body: body, button: button, closeButton: closeButton) + setDefaults() } - open override func setDefaults() { - super.setDefaults() + open func setDefaults() { if topLabel.numberOfLines == nil { topLabel.numberOfLines = 1 } - if topLabel.textColor == nil { - switch style { - case .error, .warning: - topLabel.textColor = Color(uiColor: .mvmBlack) - default: - topLabel.textColor = Color(uiColor: .mvmWhite) - } - } if topLabel.textAlignment == nil { topLabel.textAlignment = .center } @@ -69,6 +61,7 @@ open class CollapsableNotificationModel: NotificationMoleculeModel { self.initiallyCollapsed = initiallyCollapsed } try super.init(from: decoder) + setDefaults() } open override func encode(to encoder: Encoder) throws { diff --git a/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotificationTopView.swift b/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotificationTopView.swift index 01c0b3d5..1a9512d1 100644 --- a/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotificationTopView.swift +++ b/MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotificationTopView.swift @@ -52,7 +52,7 @@ import Foundation open override func reset() { super.reset() label.setFontStyle(.BoldBodySmall) - label.textColor = .white + label.textColor = .black label.textAlignment = .center } diff --git a/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationMoleculeModel.swift b/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationMoleculeModel.swift index 9958b59a..0fc5d5d5 100644 --- a/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationMoleculeModel.swift +++ b/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationMoleculeModel.swift @@ -5,9 +5,9 @@ // Created by Scott Pfeil on 9/15/20. // Copyright © 2020 Verizon Wireless. All rights reserved. // +import VDS - -open class NotificationMoleculeModel: ContainerModel, MoleculeModelProtocol { +open class NotificationMoleculeModel: MoleculeModelProtocol { /** The style of the notification: @@ -21,91 +21,48 @@ open class NotificationMoleculeModel: ContainerModel, MoleculeModelProtocol { case error case warning case information + + var toVDSStyle: VDS.Notification.Style { + switch self { + case .success: + .success + case .error: + .error + case .warning: + .warning + case .information: + .info + } + } } //-------------------------------------------------- // MARK: - Properties //-------------------------------------------------- - + public var id: String = UUID().uuidString public class var identifier: String { "notification" } public var accessibilityIdentifier: String? public var backgroundColor: Color? public var headline: LabelModel public var body: LabelModel? public var button: ButtonModel? + public var secondaryButton: ButtonModel? public var closeButton: NotificationXButtonModel? - public var style: NotificationMoleculeModel.Style = .success + public var style: Style = .success + public var inverted: Bool = false //-------------------------------------------------- // MARK: - Initializer //-------------------------------------------------- - public init(with headline: LabelModel, style: NotificationMoleculeModel.Style = .success, backgroundColor: Color? = nil, body: LabelModel? = nil, button: ButtonModel? = nil, closeButton: NotificationXButtonModel? = nil) { + public init(with headline: LabelModel, style: NotificationMoleculeModel.Style = .success, backgroundColor: Color? = nil, body: LabelModel? = nil, button: ButtonModel? = nil, secondaryButton: ButtonModel? = nil, closeButton: NotificationXButtonModel? = nil) { self.headline = headline self.style = style self.backgroundColor = backgroundColor self.body = body self.button = button + self.secondaryButton = secondaryButton self.closeButton = closeButton - super.init() - } - - //-------------------------------------------------- - // MARK: - Default - //-------------------------------------------------- - - open override func setDefaults() { - useHorizontalMargins = true - useVerticalMargins = true - topPadding = PaddingTwo - bottomPadding = PaddingTwo - - if backgroundColor == nil { - switch style { - case .error: - backgroundColor = Color(uiColor: .mvmOrange) - case .warning: - backgroundColor = Color(uiColor: .mvmYellow) - case .information: - backgroundColor = Color(uiColor: .mvmBlue) - default: - backgroundColor = Color(uiColor: .mvmGreen) - } - } - if headline.textColor == nil { - switch style { - case .error, .warning: - headline.textColor = Color(uiColor: .mvmBlack) - default: - headline.textColor = Color(uiColor: .mvmWhite) - } - } - if body?.textColor == nil { - switch style { - case .error, .warning: - body?.textColor = Color(uiColor: .mvmBlack) - default: - body?.textColor = Color(uiColor: .mvmWhite) - } - } - - button?.size = .small - button?.style = .secondary - switch style { - case .error, .warning: - button?.inverted = false - default: - button?.inverted = true - } - - if closeButton?.color == nil { - switch style { - case .error, .warning: - closeButton?.color = Color(uiColor: .mvmBlack) - default: - closeButton?.color = Color(uiColor: .mvmWhite) - } - } } //-------------------------------------------------- @@ -119,7 +76,9 @@ open class NotificationMoleculeModel: ContainerModel, MoleculeModelProtocol { case headline case body case button + case secondaryButton case closeButton + case inverted case style } @@ -129,27 +88,34 @@ open class NotificationMoleculeModel: ContainerModel, MoleculeModelProtocol { required public init(from decoder: Decoder) throws { let typeContainer = try decoder.container(keyedBy: CodingKeys.self) - backgroundColor = try typeContainer.decodeIfPresent(Color.self, forKey: .backgroundColor) accessibilityIdentifier = try typeContainer.decodeIfPresent(String.self, forKey: .accessibilityIdentifier) headline = try typeContainer.decode(LabelModel.self, forKey: .headline) body = try typeContainer.decodeIfPresent(LabelModel.self, forKey: .body) button = try typeContainer.decodeIfPresent(ButtonModel.self, forKey: .button) + secondaryButton = try typeContainer.decodeIfPresent(ButtonModel.self, forKey: .secondaryButton) closeButton = try typeContainer.decodeIfPresent(NotificationXButtonModel.self, forKey: .closeButton) + inverted = try typeContainer.decodeIfPresent(Bool.self, forKey: .inverted) ?? false if let style = try typeContainer.decodeIfPresent(NotificationMoleculeModel.Style.self, forKey: .style) { self.style = style } - super.init() } - open override func encode(to encoder: Encoder) throws { + open func encode(to encoder: Encoder) throws { var container = encoder.container(keyedBy: CodingKeys.self) try container.encode(moleculeName, forKey: .moleculeName) - try container.encodeIfPresent(backgroundColor, forKey: .backgroundColor) try container.encodeIfPresent(accessibilityIdentifier, forKey: .accessibilityIdentifier) try container.encode(headline, forKey: .headline) try container.encodeIfPresent(body, forKey: .body) try container.encodeIfPresent(button, forKey: .button) + try container.encodeIfPresent(secondaryButton, forKey: .secondaryButton) try container.encodeIfPresent(closeButton, forKey: .closeButton) + try container.encodeIfPresent(inverted, forKey: .inverted) try container.encode(style, forKey: .style) } } + +extension NotificationMoleculeModel { + public var surface: Surface { + inverted ? .dark : .light + } +} diff --git a/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationMoleculeView.swift b/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationMoleculeView.swift index 200dea16..38c58aa9 100644 --- a/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationMoleculeView.swift +++ b/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationMoleculeView.swift @@ -7,90 +7,101 @@ // import Foundation - -@objcMembers open class NotificationMoleculeView: Container { +import VDS +@objcMembers open class NotificationMoleculeView: VDS.Notification, VDSMoleculeViewProtocol { + + + //-------------------------------------------------- + // MARK: - Properties + //-------------------------------------------------- + + open var viewModel: NotificationMoleculeModel! + public var delegateObject: MVMCoreUIDelegateObject? + public var additionalData: [AnyHashable: Any]? + + //-------------------------------------------------- + // MARK: - VDSMoleculeViewProtocol + //-------------------------------------------------- + + open func viewModelDidUpdate() { + surface = viewModel.surface + title = viewModel.headline.text + subTitle = viewModel.body?.text + + if let button = viewModel.button { + primaryButtonModel = .init(text: button.title, onClick: {[weak self] _ in + guard let self else { return } + self.executeAction(model: button, delegateObject: self.delegateObject, additionalData: self.additionalData) + }) + } + + if let secondaryButton = viewModel.secondaryButton { + secondaryButtonModel = .init(text: secondaryButton.title, onClick: {[weak self] _ in + guard let self else { return } + self.executeAction(model: secondaryButton, delegateObject: self.delegateObject, additionalData: self.additionalData) + }) + } + + if let accessibilityIdentifier = viewModel.accessibilityIdentifier { + self.accessibilityIdentifier = accessibilityIdentifier + } + + if let closeButton = viewModel.closeButton { + onCloseClick = { [weak self] _ in + guard let self else { return } + self.executeAction(model: closeButton, delegateObject: self.delegateObject, additionalData: self.additionalData) } + } + + hideCloseButton = viewModel.closeButton == nil + style = viewModel.style.toVDSStyle + } + //-------------------------------------------------- // MARK: - Outlets //-------------------------------------------------- - - public let headline = Label(fontStyle: .BoldBodySmall) - public let body = Label(fontStyle: .RegularBodySmall) - public let button = PillButton() - public let closeButton = NotificationXButton() - public var labelStack: Stack! - public var horizontalStack: Stack! - // Legacy constant private static let viewHeight: CGFloat = 96.0 //-------------------------------------------------- // MARK: - Life Cycle //-------------------------------------------------- - - public override func setupView() { - super.setupView() - reset() - - // Buttons should have highest priority, then headline, then body - headline.setContentCompressionResistancePriority(UILayoutPriority(rawValue: 500), for: .horizontal) - headline.setContentHuggingPriority(.required, for: .vertical) - body.setContentCompressionResistancePriority(UILayoutPriority(rawValue: 500), for: .horizontal) - body.setContentHuggingPriority(.required, for: .vertical) - headline.setContentCompressionResistancePriority(UILayoutPriority(rawValue: body.contentCompressionResistancePriority(for: .vertical).rawValue + 40), for: .vertical) - headline.lineBreakMode = .byTruncatingTail - body.lineBreakMode = .byTruncatingTail - button.setContentCompressionResistancePriority(.defaultHigh, for: .horizontal) - - labelStack = Stack.createStack(with: [headline, body], spacing: 0) - horizontalStack = Stack.createStack(with: [(view: labelStack, model: StackItemModel()),(view: button, model: StackItemModel(horizontalAlignment: .fill)),(view: closeButton, model: StackItemModel(horizontalAlignment: .fill))], axis: .horizontal) - addAndContain(horizontalStack) - labelStack.restack() - horizontalStack.restack() - - heightAnchor.constraint(equalToConstant: Self.viewHeight).isActive = true - } - - open override func reset() { - super.reset() - backgroundColor = .mvmGreen() - headline.textColor = .white - body.textColor = .white + open override func updateAccessibility() { + super.updateAccessibility() + Self.amendAccesibilityLabel(for: titleLabel) + Self.amendAccesibilityLabel(for: subTitleLabel) + Self.amendAccesibilityLabel(for: primaryButton) + Self.amendAccesibilityLabel(for: secondaryButton) + Self.amendAccesibilityLabel(for: closeButton) } //-------------------------------------------------- // MARK: - Molecule //-------------------------------------------------- - - open override func set(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?, _ additionalData: [AnyHashable: Any]?) { - super.set(with: model, delegateObject, additionalData) - guard let model = model as? NotificationMoleculeModel else { return } - labelStack.updateContainedMolecules(with: [model.headline, model.body], delegateObject, nil) - horizontalStack.updateContainedMolecules(with: [labelStack.stackModel, model.button, model.closeButton], delegateObject, nil) - updateAccessibility() - } - - open override class func estimatedHeight(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?) -> CGFloat? { + public func updateView(_ size: CGFloat) { } + + open class func estimatedHeight(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?) -> CGFloat? { return viewHeight } - - open func updateAccessibility() { - NotificationMoleculeView.amendAccesibilityLabel(for: headline) - NotificationMoleculeView.amendAccesibilityLabel(for: body) - NotificationMoleculeView.amendAccesibilityLabel(for: button) - NotificationMoleculeView.amendAccesibilityLabel(for: closeButton) - } - + /// Formats the accessibilityLabel so voice over users know it's in the notification. - static public func amendAccesibilityLabel(for view: UIView) { - guard let amendment = MVMCoreUIUtility.hardcodedString(withKey: "top_alert_notification"), - let accessibilityLabel = view.accessibilityLabel, - !accessibilityLabel.hasPrefix(amendment) else { return } - view.accessibilityLabel = "\(amendment) - \(accessibilityLabel)" + public class func amendAccesibilityLabel(for view: UIView?) { + guard let view, + let amendment = MVMCoreUIUtility.hardcodedString(withKey: "top_alert_notification") + else { return } + view.amendAccesibilityLabel(with: amendment) } } extension NotificationMoleculeView: AccessibilityProtocol { public func getAccessibilityLayoutChangedArgument() -> Any? { - return headline + return titleLabel + } +} + +extension UIView { + /// Formats the accessibilityLabel so voice over users know it's in the notification. + public func amendAccesibilityLabel(with amendment: String) { + guard let accessibilityLabel, !accessibilityLabel.hasPrefix(amendment) else { return } + self.accessibilityLabel = "\(amendment) - \(accessibilityLabel)" } } diff --git a/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationXButton.swift b/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationXButton.swift index d0071668..c102d064 100644 --- a/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationXButton.swift +++ b/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationXButton.swift @@ -30,7 +30,6 @@ import MVMCore open override func set(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?, _ additionalData: [AnyHashable : Any]?) { super.set(with: model, delegateObject, additionalData) guard let model = model as? NotificationXButtonModel else { return } - tintColor = model.color?.uiColor ?? .white // TODO: Temporary, consider action for dismissing top alert if model.action.actionType == ActionNoopModel.identifier { diff --git a/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationXButtonModel.swift b/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationXButtonModel.swift index ac1c8905..0206b79d 100644 --- a/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationXButtonModel.swift +++ b/MVMCoreUI/Atomic/Molecules/TopNotification/NotificationXButtonModel.swift @@ -15,25 +15,21 @@ public class NotificationXButtonModel: ButtonModelProtocol, MoleculeModelProtoco public var id: String = UUID().uuidString public var backgroundColor: Color? - public var color: Color? public var action: ActionModelProtocol = ActionNoopModel() private enum CodingKeys: String, CodingKey { case id case moleculeName - case color case action } - public init(color: Color? = nil, action: ActionModelProtocol = ActionNoopModel()) { - self.color = color + public init(action: ActionModelProtocol = ActionNoopModel()) { self.action = action } public required init(from decoder: Decoder) throws { let typeContainer = try decoder.container(keyedBy: CodingKeys.self) id = try typeContainer.decodeIfPresent(String.self, forKey: .id) ?? UUID().uuidString - color = try typeContainer.decodeIfPresent(Color.self, forKey: .color) if let action: ActionModelProtocol = try typeContainer.decodeModelIfPresent(codingKey: .action) { self.action = action } @@ -43,7 +39,6 @@ public class NotificationXButtonModel: ButtonModelProtocol, MoleculeModelProtoco var container = encoder.container(keyedBy: CodingKeys.self) try container.encode(id, forKey: .id) try container.encode(moleculeName, forKey: .moleculeName) - try container.encodeIfPresent(color, forKey: .color) try container.encodeModel(action, forKey: .action) } } diff --git a/MVMCoreUI/Atomic/Protocols/MoleculeViewProtocol.swift b/MVMCoreUI/Atomic/Protocols/MoleculeViewProtocol.swift index c3c07b6a..3869b398 100644 --- a/MVMCoreUI/Atomic/Protocols/MoleculeViewProtocol.swift +++ b/MVMCoreUI/Atomic/Protocols/MoleculeViewProtocol.swift @@ -66,6 +66,14 @@ extension MoleculeViewProtocol { set(with: model, delegateObject, additionalData) } } + + public func executeAction(model: T, delegateObject: MVMCoreUIDelegateObject?, additionalData: [AnyHashable: Any]?) { + Task(priority: .userInitiated) { + try await (delegateObject?.actionDelegate as? ActionDelegateProtocol)?.performAction(with: model.action, + additionalData: MVMCoreUIActionHandler.add(sourceModel: model, to: additionalData), + delegateObject: delegateObject) + } + } } // Convenience Functions diff --git a/MVMCoreUI/Atomic/Protocols/VDSMoleculeViewProtocol.swift b/MVMCoreUI/Atomic/Protocols/VDSMoleculeViewProtocol.swift index 1bcc78c6..4bebf0c7 100644 --- a/MVMCoreUI/Atomic/Protocols/VDSMoleculeViewProtocol.swift +++ b/MVMCoreUI/Atomic/Protocols/VDSMoleculeViewProtocol.swift @@ -21,7 +21,11 @@ public protocol VDSMoleculeViewProtocol: MoleculeViewProtocol, MVMCoreViewProtoc } extension VDSMoleculeViewProtocol { - + public var model: MoleculeModelProtocol { + get { viewModel } + set { } + } + public func set(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?, _ additionalData: [AnyHashable: Any]?) { self.model = model guard let castedModel = model as? ViewModel else { return } diff --git a/MVMCoreUI/Categories/UIColor+Extension.swift b/MVMCoreUI/Categories/UIColor+Extension.swift index 78363630..f4ef0f7f 100644 --- a/MVMCoreUI/Categories/UIColor+Extension.swift +++ b/MVMCoreUI/Categories/UIColor+Extension.swift @@ -56,6 +56,18 @@ extension UIColor { "upGold2": (.vzupGold2, "#F4CA53"), "upGold3": (.vzupGold3, "#CC9B2D")] + //-------------------------------------------------- + // MARK: - Helper + //-------------------------------------------------- + public var rgbComponents: (red: CGFloat, green: CGFloat, blue: CGFloat, alpha: CGFloat) { + var red: CGFloat = 0 + var green: CGFloat = 0 + var blue: CGFloat = 0 + var alpha: CGFloat = 0 + getRed(&red, green: &green, blue: &blue, alpha: &alpha) + return (red, green, blue, alpha) + } + //-------------------------------------------------- // MARK: - Brand //--------------------------------------------------