Fixed: duplicate calls to scrollRangeToVisible

KennethTsang · KennethTsang · commit 723a11b10b02 · 2017-11-01T00:14:32.000+08:00
diff --git a/GrowingTextView.podspec b/GrowingTextView.podspec
@@ -8,7 +8,7 @@
 
 Pod::Spec.new do |s|
   s.name             = "GrowingTextView"
-  s.version          = "0.5.1"
+  s.version          = "0.5.2"
   s.summary          = "UITextView on Swift 3 and Swift 4. Support auto growing, placeholder and length limit."
 
 # This description is used to generate tags and improve search results.
diff --git a/Pod/Classes/GrowingTextView.swift b/Pod/Classes/GrowingTextView.swift
@@ -135,8 +135,9 @@ open class GrowingTextView: UITextView {
                 // Workaround to for incorrect scroll position on Swift4
                 self.heightConstraint!.constant += 0.0000001
                 self.isGrowing = false
+            } else {
+                self.scrollRangeToVisible(NSMakeRange(-1, 0)) // Scroll to bottom
             }
-            self.scrollRangeToVisible(NSMakeRange(-1, 0)) // Scroll to bottom
         } else {
             self.scrollRangeToVisible(NSMakeRange(0, 0)) // Scroll to top
         }

Original file line number	Diff line number	Diff line change
`@@ -135,8 +135,9 @@ open class GrowingTextView: UITextView {`
`135`	`135`	`// Workaround to for incorrect scroll position on Swift4`
`136`	`136`	`self.heightConstraint!.constant += 0.0000001`
`137`	`137`	`self.isGrowing = false`
	`138`	`+ } else {`
	`139`	`+ self.scrollRangeToVisible(NSMakeRange(-1, 0)) // Scroll to bottom`
`138`	`140`	`}`
`139`		`- self.scrollRangeToVisible(NSMakeRange(-1, 0)) // Scroll to bottom`
`140`	`141`	`} else {`
`141`	`142`	`self.scrollRangeToVisible(NSMakeRange(0, 0)) // Scroll to top`
`142`	`143`	`}`