Commit 2e262c5a by Daniel Dahan

Revert "development: updated Pulse internal API"

This reverts commit 777f4942.
parent 777f4942
...@@ -49,8 +49,7 @@ open class Bar: View { ...@@ -49,8 +49,7 @@ open class Bar: View {
/// Should center the contentView. /// Should center the contentView.
open var contentViewAlignment = ContentViewAlignment.full { open var contentViewAlignment = ContentViewAlignment.full {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -101,8 +100,7 @@ open class Bar: View { ...@@ -101,8 +100,7 @@ open class Bar: View {
open var gridFactor: CGFloat = 12 { open var gridFactor: CGFloat = 12 {
didSet { didSet {
assert(0 < gridFactor, "[Material Error: gridFactor must be greater than 0.]") assert(0 < gridFactor, "[Material Error: gridFactor must be greater than 0.]")
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -115,8 +113,7 @@ open class Bar: View { ...@@ -115,8 +113,7 @@ open class Bar: View {
for v in oldValue { for v in oldValue {
v.removeFromSuperview() v.removeFromSuperview()
} }
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -126,8 +123,7 @@ open class Bar: View { ...@@ -126,8 +123,7 @@ open class Bar: View {
for v in oldValue { for v in oldValue {
v.removeFromSuperview() v.removeFromSuperview()
} }
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -44,7 +44,7 @@ open class Button: UIButton, Pulseable, PulseableLayer { ...@@ -44,7 +44,7 @@ open class Button: UIButton, Pulseable, PulseableLayer {
/// A reference to the pulse layer. /// A reference to the pulse layer.
internal var pulseLayer: CALayer? { internal var pulseLayer: CALayer? {
return pulse.layer return pulse.pulseLayer
} }
/// PulseAnimation value. /// PulseAnimation value.
...@@ -263,7 +263,7 @@ extension Button { ...@@ -263,7 +263,7 @@ extension Button {
/// Prepares the pulse motion. /// Prepares the pulse motion.
fileprivate func preparePulse() { fileprivate func preparePulse() {
pulse = Pulse(view: self, layer: visualLayer) pulse = Pulse(pulseView: self, pulseLayer: visualLayer)
} }
/** /**
......
...@@ -69,8 +69,7 @@ open class Card: PulseView { ...@@ -69,8 +69,7 @@ open class Card: PulseView {
if let v = toolbar { if let v = toolbar {
container.addSubview(v) container.addSubview(v)
} }
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -85,8 +84,7 @@ open class Card: PulseView { ...@@ -85,8 +84,7 @@ open class Card: PulseView {
@IBInspectable @IBInspectable
open var toolbarEdgeInsets = EdgeInsets.zero { open var toolbarEdgeInsets = EdgeInsets.zero {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -99,8 +97,7 @@ open class Card: PulseView { ...@@ -99,8 +97,7 @@ open class Card: PulseView {
v.clipsToBounds = true v.clipsToBounds = true
container.addSubview(v) container.addSubview(v)
} }
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -115,8 +112,7 @@ open class Card: PulseView { ...@@ -115,8 +112,7 @@ open class Card: PulseView {
@IBInspectable @IBInspectable
open var contentViewEdgeInsets = EdgeInsets.zero { open var contentViewEdgeInsets = EdgeInsets.zero {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -128,8 +124,7 @@ open class Card: PulseView { ...@@ -128,8 +124,7 @@ open class Card: PulseView {
if let v = bottomBar { if let v = bottomBar {
container.addSubview(v) container.addSubview(v)
} }
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -144,8 +139,7 @@ open class Card: PulseView { ...@@ -144,8 +139,7 @@ open class Card: PulseView {
@IBInspectable @IBInspectable
open var bottomBarEdgeInsets = EdgeInsets.zero { open var bottomBarEdgeInsets = EdgeInsets.zero {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -45,7 +45,7 @@ open class CollectionReusableView: UICollectionReusableView, Pulseable, Pulseabl ...@@ -45,7 +45,7 @@ open class CollectionReusableView: UICollectionReusableView, Pulseable, Pulseabl
/// A reference to the pulse layer. /// A reference to the pulse layer.
internal var pulseLayer: CALayer? { internal var pulseLayer: CALayer? {
return pulse.layer return pulse.pulseLayer
} }
/// PulseAnimation value. /// PulseAnimation value.
...@@ -301,7 +301,7 @@ open class CollectionReusableView: UICollectionReusableView, Pulseable, Pulseabl ...@@ -301,7 +301,7 @@ open class CollectionReusableView: UICollectionReusableView, Pulseable, Pulseabl
extension CollectionReusableView { extension CollectionReusableView {
/// Prepares the pulse motion. /// Prepares the pulse motion.
fileprivate func preparePulse() { fileprivate func preparePulse() {
pulse = Pulse(view: self, layer: visualLayer) pulse = Pulse(pulseView: self, pulseLayer: visualLayer)
pulseAnimation = .none pulseAnimation = .none
} }
......
...@@ -45,7 +45,7 @@ open class CollectionViewCell: UICollectionViewCell, Pulseable, PulseableLayer { ...@@ -45,7 +45,7 @@ open class CollectionViewCell: UICollectionViewCell, Pulseable, PulseableLayer {
/// A reference to the pulse layer. /// A reference to the pulse layer.
internal var pulseLayer: CALayer? { internal var pulseLayer: CALayer? {
return pulse.layer return pulse.pulseLayer
} }
/// PulseAnimation value. /// PulseAnimation value.
...@@ -259,7 +259,7 @@ open class CollectionViewCell: UICollectionViewCell, Pulseable, PulseableLayer { ...@@ -259,7 +259,7 @@ open class CollectionViewCell: UICollectionViewCell, Pulseable, PulseableLayer {
extension CollectionViewCell { extension CollectionViewCell {
/// Prepares the pulse motion. /// Prepares the pulse motion.
fileprivate func preparePulse() { fileprivate func preparePulse() {
pulse = Pulse(view: self, layer: visualLayer) pulse = Pulse(pulseView: self, pulseLayer: visualLayer)
} }
/// Prepares the visualLayer property. /// Prepares the visualLayer property.
......
...@@ -36,8 +36,7 @@ open class ErrorTextField: TextField { ...@@ -36,8 +36,7 @@ open class ErrorTextField: TextField {
open var isErrorRevealed = false { open var isErrorRevealed = false {
didSet { didSet {
detailLabel.isHidden = !isErrorRevealed detailLabel.isHidden = !isErrorRevealed
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -67,8 +67,7 @@ open class FABMenuItem: View { ...@@ -67,8 +67,7 @@ open class FABMenuItem: View {
} }
set(value) { set(value) {
titleLabel.text = value titleLabel.text = value
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -201,8 +200,7 @@ open class FABMenu: View { ...@@ -201,8 +200,7 @@ open class FABMenu: View {
spring.springDirection = .right spring.springDirection = .right
} }
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -44,8 +44,7 @@ open class ImageCard: Card { ...@@ -44,8 +44,7 @@ open class ImageCard: Card {
*/ */
open var imageViewDisplay = Display.partial { open var imageViewDisplay = Display.partial {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -60,8 +59,7 @@ open class ImageCard: Card { ...@@ -60,8 +59,7 @@ open class ImageCard: Card {
@IBInspectable @IBInspectable
open var imageViewEdgeInsets = EdgeInsets.zero { open var imageViewEdgeInsets = EdgeInsets.zero {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -73,16 +71,14 @@ open class ImageCard: Card { ...@@ -73,16 +71,14 @@ open class ImageCard: Card {
if let v = imageView { if let v = imageView {
container.addSubview(v) container.addSubview(v)
} }
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
/// An ImageCardToolbarAlignment value. /// An ImageCardToolbarAlignment value.
open var toolbarAlignment = ToolbarAlignment.bottom { open var toolbarAlignment = ToolbarAlignment.bottom {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -51,8 +51,7 @@ open class NavigationBar: UINavigationBar { ...@@ -51,8 +51,7 @@ open class NavigationBar: UINavigationBar {
@IBInspectable @IBInspectable
open var contentEdgeInsets = EdgeInsets.zero { open var contentEdgeInsets = EdgeInsets.zero {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -67,8 +66,7 @@ open class NavigationBar: UINavigationBar { ...@@ -67,8 +66,7 @@ open class NavigationBar: UINavigationBar {
@IBInspectable @IBInspectable
open var interimSpace: InterimSpace = 0 { open var interimSpace: InterimSpace = 0 {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -77,8 +75,7 @@ open class NavigationBar: UINavigationBar { ...@@ -77,8 +75,7 @@ open class NavigationBar: UINavigationBar {
open var gridFactor: CGFloat = 12 { open var gridFactor: CGFloat = 12 {
didSet { didSet {
assert(0 < gridFactor, "[Material Error: gridFactor must be greater than 0.]") assert(0 < gridFactor, "[Material Error: gridFactor must be greater than 0.]")
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -38,8 +38,7 @@ public class NavigationItem: NSObject { ...@@ -38,8 +38,7 @@ public class NavigationItem: NSObject {
/// Should center the contentView. /// Should center the contentView.
open var contentViewAlignment = ContentViewAlignment.center { open var contentViewAlignment = ContentViewAlignment.center {
didSet { didSet {
navigationBar?.setNeedsLayout() navigationBar?.layoutSubviews()
navigationBar?.layoutIfNeeded()
} }
} }
...@@ -61,8 +60,7 @@ public class NavigationItem: NSObject { ...@@ -61,8 +60,7 @@ public class NavigationItem: NSObject {
for v in oldValue { for v in oldValue {
v.removeFromSuperview() v.removeFromSuperview()
} }
navigationBar?.setNeedsLayout() navigationBar?.layoutSubviews()
navigationBar?.layoutIfNeeded()
} }
} }
...@@ -72,8 +70,7 @@ public class NavigationItem: NSObject { ...@@ -72,8 +70,7 @@ public class NavigationItem: NSObject {
for v in oldValue { for v in oldValue {
v.removeFromSuperview() v.removeFromSuperview()
} }
navigationBar?.setNeedsLayout() navigationBar?.layoutSubviews()
navigationBar?.layoutIfNeeded()
} }
} }
...@@ -112,8 +109,7 @@ public class NavigationItem: NSObject { ...@@ -112,8 +109,7 @@ public class NavigationItem: NSObject {
/// Reloads the subviews for the NavigationBar. /// Reloads the subviews for the NavigationBar.
internal func reload() { internal func reload() {
navigationBar?.setNeedsLayout() navigationBar?.layoutSubviews()
navigationBar?.layoutIfNeeded()
} }
/// Prepares the titleLabel. /// Prepares the titleLabel.
......
...@@ -42,8 +42,7 @@ open class PresenterCard: Card { ...@@ -42,8 +42,7 @@ open class PresenterCard: Card {
@IBInspectable @IBInspectable
open var presenterViewEdgeInsets = EdgeInsets.zero { open var presenterViewEdgeInsets = EdgeInsets.zero {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -56,8 +55,7 @@ open class PresenterCard: Card { ...@@ -56,8 +55,7 @@ open class PresenterCard: Card {
v.clipsToBounds = true v.clipsToBounds = true
container.addSubview(v) container.addSubview(v)
} }
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -60,10 +60,10 @@ internal protocol PulseableLayer { ...@@ -60,10 +60,10 @@ internal protocol PulseableLayer {
public struct Pulse { public struct Pulse {
/// A UIView that is Pulseable. /// A UIView that is Pulseable.
fileprivate weak var view: UIView? fileprivate weak var pulseView: UIView?
/// The layer the pulse layers are added to. /// The layer the pulse layers are added to.
internal weak var layer: CALayer? internal weak var pulseLayer: CALayer?
/// Pulse layers. /// Pulse layers.
fileprivate var layers = [CAShapeLayer]() fileprivate var layers = [CAShapeLayer]()
...@@ -79,12 +79,12 @@ public struct Pulse { ...@@ -79,12 +79,12 @@ public struct Pulse {
/** /**
An initializer that takes a given view and pulse layer. An initializer that takes a given view and pulse layer.
- Parameter view: An optional UIView. - Parameter pulseView: An optional UIView.
- Parameter layer: An optional CALayer. - Parameter pulseLayer: An optional CALayer.
*/ */
internal init(view: UIView?, layer: CALayer?) { internal init(pulseView: UIView?, pulseLayer: CALayer?) {
self.view = view self.pulseView = pulseView
self.layer = layer self.pulseLayer = pulseLayer
} }
/** /**
...@@ -92,11 +92,11 @@ public struct Pulse { ...@@ -92,11 +92,11 @@ public struct Pulse {
- Parameter point: A point to pulse from. - Parameter point: A point to pulse from.
*/ */
public mutating func expand(point: CGPoint) { public mutating func expand(point: CGPoint) {
guard let v = view else { guard let view = pulseView else {
return return
} }
guard let layer = layer else { guard let layer = pulseLayer else {
return return
} }
...@@ -116,8 +116,8 @@ public struct Pulse { ...@@ -116,8 +116,8 @@ public struct Pulse {
layer.masksToBounds = !(.centerRadialBeyondBounds == animation || .radialBeyondBounds == animation) layer.masksToBounds = !(.centerRadialBeyondBounds == animation || .radialBeyondBounds == animation)
let w = v.bounds.width let w = view.bounds.width
let h = v.bounds.height let h = view.bounds.height
Motion.disable({ [ Motion.disable({ [
n = .center == animation ? w < h ? w : h : w < h ? h : w, n = .center == animation ? w < h ? w : h : w < h ? h : w,
......
...@@ -36,7 +36,7 @@ open class PulseView: View, Pulseable, PulseableLayer { ...@@ -36,7 +36,7 @@ open class PulseView: View, Pulseable, PulseableLayer {
/// A reference to the pulse layer. /// A reference to the pulse layer.
internal var pulseLayer: CALayer? { internal var pulseLayer: CALayer? {
return pulse.layer return pulse.pulseLayer
} }
/// PulseAnimation value. /// PulseAnimation value.
...@@ -132,6 +132,6 @@ open class PulseView: View, Pulseable, PulseableLayer { ...@@ -132,6 +132,6 @@ open class PulseView: View, Pulseable, PulseableLayer {
extension PulseView { extension PulseView {
/// Prepares the pulse motion. /// Prepares the pulse motion.
fileprivate func preparePulse() { fileprivate func preparePulse() {
pulse = Pulse(view: self, layer: visualLayer) pulse = Pulse(pulseView: self, pulseLayer: visualLayer)
} }
} }
...@@ -44,8 +44,7 @@ open class Snackbar: Bar { ...@@ -44,8 +44,7 @@ open class Snackbar: Bar {
} }
set(value) { set(value) {
textLabel.text = value textLabel.text = value
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -118,8 +118,7 @@ open class TabBar: Bar { ...@@ -118,8 +118,7 @@ open class TabBar: Bar {
centerViews = buttons as [UIView] centerViews = buttons as [UIView]
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -153,8 +152,7 @@ open class TabBar: Bar { ...@@ -153,8 +152,7 @@ open class TabBar: Bar {
/// A value for the line alignment. /// A value for the line alignment.
open var lineAlignment = TabBarLineAlignment.bottom { open var lineAlignment = TabBarLineAlignment.bottom {
didSet { didSet {
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -44,7 +44,7 @@ open class TableViewCell: UITableViewCell, Pulseable, PulseableLayer { ...@@ -44,7 +44,7 @@ open class TableViewCell: UITableViewCell, Pulseable, PulseableLayer {
/// A reference to the pulse layer. /// A reference to the pulse layer.
internal var pulseLayer: CALayer? { internal var pulseLayer: CALayer? {
return pulse.layer return pulse.pulseLayer
} }
/// PulseAnimation value. /// PulseAnimation value.
...@@ -180,7 +180,7 @@ open class TableViewCell: UITableViewCell, Pulseable, PulseableLayer { ...@@ -180,7 +180,7 @@ open class TableViewCell: UITableViewCell, Pulseable, PulseableLayer {
extension TableViewCell { extension TableViewCell {
/// Prepares the pulse motion. /// Prepares the pulse motion.
fileprivate func preparePulse() { fileprivate func preparePulse() {
pulse = Pulse(view: self, layer: visualLayer) pulse = Pulse(pulseView: self, pulseLayer: visualLayer)
} }
/// Prepares the visualLayer property. /// Prepares the visualLayer property.
......
...@@ -88,8 +88,7 @@ open class TextField: UITextField { ...@@ -88,8 +88,7 @@ open class TextField: UITextField {
open override var leftView: UIView? { open override var leftView: UIView? {
didSet { didSet {
prepareLeftView() prepareLeftView()
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -186,8 +185,7 @@ open class TextField: UITextField { ...@@ -186,8 +185,7 @@ open class TextField: UITextField {
} }
set(value) { set(value) {
placeholderLabel.text = value placeholderLabel.text = value
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -227,8 +225,7 @@ open class TextField: UITextField { ...@@ -227,8 +225,7 @@ open class TextField: UITextField {
} }
set(value) { set(value) {
detailLabel.text = value detailLabel.text = value
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -288,8 +285,7 @@ open class TextField: UITextField { ...@@ -288,8 +285,7 @@ open class TextField: UITextField {
rightView = clearIconButton rightView = clearIconButton
isClearIconButtonAutoHandled = isClearIconButtonAutoHandled ? true : false isClearIconButtonAutoHandled = isClearIconButtonAutoHandled ? true : false
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -336,8 +332,7 @@ open class TextField: UITextField { ...@@ -336,8 +332,7 @@ open class TextField: UITextField {
rightView = visibilityIconButton rightView = visibilityIconButton
isVisibilityIconButtonAutoHandled = isVisibilityIconButtonAutoHandled ? true : false isVisibilityIconButtonAutoHandled = isVisibilityIconButtonAutoHandled ? true : false
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
...@@ -41,8 +41,7 @@ open class Toolbar: Bar { ...@@ -41,8 +41,7 @@ open class Toolbar: Bar {
} }
set(value) { set(value) {
titleLabel.text = value titleLabel.text = value
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
...@@ -58,8 +57,7 @@ open class Toolbar: Bar { ...@@ -58,8 +57,7 @@ open class Toolbar: Bar {
} }
set(value) { set(value) {
detailLabel.text = value detailLabel.text = value
setNeedsLayout() layoutSubviews()
layoutIfNeeded()
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment