Change scroll enabled (#347)

* Allow changing allowAutogrowing after init

* Recalculate editor height on setting back scrollable

Author: rajdeep

Diff for: Proton/Sources/Swift/Base/AutogrowingTextView.swift

@@ -27,22 +27,15 @@ class AutogrowingTextView: UITextView {
     private var allowAutogrowing: Bool
     weak var boundsObserver: BoundsObserving?
     private var maxHeightConstraint: NSLayoutConstraint!
-    private var heightAnchorConstraint: NSLayoutConstraint!
+    private var heightAnchorConstraint: NSLayoutConstraint?
     private var isSizeRecalculationRequired = true
 
     init(frame: CGRect = .zero, textContainer: NSTextContainer? = nil, allowAutogrowing: Bool = false) {
         self.allowAutogrowing = allowAutogrowing
         super.init(frame: frame, textContainer: textContainer)
         isScrollEnabled = false
+        addHeightConstraint()
 
-        if allowAutogrowing {
-            heightAnchorConstraint = heightAnchor.constraint(greaterThanOrEqualToConstant: contentSize.height)
-            heightAnchorConstraint.priority = .defaultHigh
-
-            NSLayoutConstraint.activate([
-                heightAnchorConstraint
-            ])
-        }
         //TODO: enable only when line numbering is turned on
         contentMode = .redraw
     }
@@ -51,6 +44,34 @@ class AutogrowingTextView: UITextView {
         fatalError("init(coder:) has not been implemented")
     }
 
+    func setAutogrowing(_ isAutogrowing: Bool) {
+        allowAutogrowing = isAutogrowing
+
+        if allowAutogrowing {
+            if heightAnchorConstraint == nil {
+                addHeightConstraint()
+                recalculateHeight()
+            }
+        }  else {
+            isScrollEnabled = false
+            if let heightAnchorConstraint {
+                NSLayoutConstraint.deactivate([heightAnchorConstraint])
+            }
+            heightAnchorConstraint = nil
+        }
+    }
+
+    private func addHeightConstraint() {
+        guard allowAutogrowing else { return }
+        let heightConstraint = heightAnchor.constraint(greaterThanOrEqualToConstant: contentSize.height)
+        heightAnchorConstraint = heightConstraint
+        heightAnchorConstraint?.priority = .defaultHigh
+
+        NSLayoutConstraint.activate([
+            heightConstraint
+        ])
+    }
+
     override func layoutSubviews() {
         super.layoutSubviews()
         guard allowAutogrowing, maxHeight != .greatestFiniteMagnitude else { return }

Diff for: Proton/Sources/Swift/Editor/EditorView.swift

@@ -1307,6 +1307,15 @@ open class EditorView: UIView {
         richTextView.recalculateHeight(size: size)
     }
 
+    /// Set the behavior for how Editor size would be updated based on content
+    /// - Parameter isAutogrowing: When `true`, uses custom calculation and constrains to size editor based on content. This is typically the case where
+    /// Editor is scrollable and needs to be confined to certain size using applied constraints. Use `false` in case Editor is itself non-scrollable but is hosted within
+    /// another scroll container. This will use iOS's internal logic for sizing the Editor based on the height of the content and is generally better performing.
+    public func setAutogrowing(_ isAutogrowing: Bool) {
+        richTextView.setAutogrowing(isAutogrowing)
+    }
+
+
     open override func canPerformAction(_ action: Selector, withSender sender: Any?) -> Bool {
         return richTextView.canPerformAction(action, withSender: sender)
     }