Merge pull request #2 from maxiru/fix/MultipleResetKeyboardSpace

SnaiNeR · web-flow · commit cfe6e039aa02 · 2020-06-10T14:26:24.000+05:00
Do not animate layout if keyboard state is not changed
diff --git a/KeyboardSpacer.js b/KeyboardSpacer.js
@@ -86,7 +86,7 @@ export default class KeyboardSpacer extends Component {
   }
 
   updateKeyboardSpace(event) {
-    if (!event.endCoordinates) {
+    if (!event.endCoordinates || this.state.isKeyboardOpened) {
       return;
     }
 
@@ -113,6 +113,10 @@ export default class KeyboardSpacer extends Component {
   }
 
   resetKeyboardSpace(event) {
+    if (!this.state.isKeyboardOpened) {
+      return;
+    }
+    
     let animationConfig = defaultAnimation;
     if (Platform.OS === 'ios') {
       animationConfig = LayoutAnimation.create(

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ export default class KeyboardSpacer extends Component {`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`updateKeyboardSpace(event) {`
`89`		`- if (!event.endCoordinates) {`
	`89`	`+ if (!event.endCoordinates \|\| this.state.isKeyboardOpened) {`
`90`	`90`	`return;`
`91`	`91`	`}`
`92`	`92`
`@@ -113,6 +113,10 @@ export default class KeyboardSpacer extends Component {`
`113`	`113`	`}`
`114`	`114`
`115`	`115`	`resetKeyboardSpace(event) {`
	`116`	`+ if (!this.state.isKeyboardOpened) {`
	`117`	`+ return;`
	`118`	`+ }`
	`119`	`+`
`116`	`120`	`let animationConfig = defaultAnimation;`
`117`	`121`	`if (Platform.OS === 'ios') {`
`118`	`122`	`animationConfig = LayoutAnimation.create(`