mbrucedogs/mvm_core_ui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

review comment

Browse Source
This commit is contained in:
Pfeil, Scott Robert 2020-09-22 09:56:31 -04:00
parent d3fd303988
commit 30924c6c50
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
MVMCoreUI/Atomic/Molecules/TopNotification/CollapsableNotification.swift
View File

@ -81,11 +81,11 @@ import Foundation
let delay: DispatchTimeInterval = DispatchTimeInterval.seconds((model as? CollapsableNotificationModel)?.collapseTime ?? 5) let delay: DispatchTimeInterval = DispatchTimeInterval.seconds((model as? CollapsableNotificationModel)?.collapseTime ?? 5)
DispatchQueue.main.asyncAfter(deadline: .now() + delay) { [weak self] in DispatchQueue.main.asyncAfter(deadline: .now() + delay) { [weak self] in
// If accessibility focused, delay collapse. // If accessibility focused, delay collapse.
guard let localSelf = self else { return } guard let self = self else { return }
if MVMCoreUIUtility.viewContainsAccessiblityFocus(localSelf) { if MVMCoreUIUtility.viewContainsAccessiblityFocus(self) {
NotificationCenter.default.addObserver(localSelf, selector: #selector(localSelf.accessibilityFocusChanged(notification:)), name: UIAccessibility.elementFocusedNotification, object: nil) NotificationCenter.default.addObserver(self, selector: #selector(self.accessibilityFocusChanged(notification:)), name: UIAccessibility.elementFocusedNotification, object: nil)
} else { } else {
localSelf.collapse() self.collapse()
} }
} }
} }
@ -150,7 +150,7 @@ import Foundation
} }
open override class func estimatedHeight(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?) -> CGFloat? { open override class func estimatedHeight(with model: MoleculeModelProtocol, _ delegateObject: MVMCoreUIDelegateObject?) -> CGFloat? {
return 96 return 120
} }
/// Collapse if focus is no longer on this top alert. /// Collapse if focus is no longer on this top alert.