Conversation
…reAnimation to animate the CALayer instead of the UIView
…ggering it through a flag
buttonScrollToBottom.heightAnchor.constraint(equalToConstant: buttonScrollToBottomSize), | ||
buttonScrollToBottom.widthAnchor.constraint(equalToConstant: buttonScrollToBottomSize), | ||
buttonScrollToBottomConstraint, | ||
buttonScrollToBottom.trailingAnchor.constraint(equalTo: view.safeAreaLayoutGuide.trailingAnchor, constant: -15) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line Length Violation: Line should be 120 characters or less: currently 123 characters (line_length)
buttonScrollToBottom.translatesAutoresizingMaskIntoConstraints = false | ||
view.addSubview(buttonScrollToBottom) | ||
view.bringSubviewToFront(buttonScrollToBottom) | ||
buttonScrollToBottomConstraint = buttonScrollToBottom.bottomAnchor.constraint(equalTo: view.bottomAnchor, constant: 50) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line Length Violation: Line should be 120 characters or less: currently 127 characters (line_length)
var keyboardHeight: CGFloat = 0 | ||
var buttonScrollToBottomConstraint: NSLayoutConstraint! | ||
var buttonScrollToBottomLayerY: CGFloat { | ||
return -composerView.layer.bounds.height - buttonScrollToBottomSize / 2 - collectionView.layoutMargins.top - keyboardHeight |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line Length Violation: Line should be 120 characters or less: currently 131 characters (line_length)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really good to me! 👍
@RocketChat/ios
NOTE: We still have some glitches, mainly when the keyboard is dismissed, this is due to the manual changes we are doing in order to manage the insets when the keyboard is being presented.