Fix scroll animations issue (#66)

* update layouts before navigating

* 2.12.2

* updated changelog

* typo

* updated changelog

Author: salvan13

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.12.2]
+### Changed
+- update layouts at the beginning of smartNavigate instead of after setFocus
+
 ## [2.12.1]
 ### Fixed
 - Fixed regression with using `autoRestoreFocus` on components that are focused + getting unmounted and don't have parent

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@noriginmedia/react-spatial-navigation",
-  "version": "2.12.1",
+  "version": "2.12.2",
   "description": "HOC-based Spatial Navigation (key navigation) solution for React",
   "main": "dist/index.js",
   "files": [

package.json

@@ -537,6 +537,10 @@ class SpatialNavigation {
     this.log('smartNavigate', 'fromParentFocusKey', fromParentFocusKey);
     this.log('smartNavigate', 'this.focusKey', this.focusKey);
 
+    if (!this.nativeMode && !fromParentFocusKey) {
+      this.updateAllLayouts();
+    }
+
     const currentComponent = this.focusableComponents[fromParentFocusKey || this.focusKey];
 
     this.log(
@@ -946,10 +950,6 @@ class SpatialNavigation {
     this.setCurrentFocusedKey(newFocusKey, details);
     this.updateParentsHasFocusedChild(newFocusKey, details);
     this.updateParentsLastFocusedChild(lastFocusedKey);
-
-    if (!this.nativeMode) {
-      this.updateAllLayouts();
-    }
   }
 
   updateAllLayouts() {