fix(community-side-navigation): fix sidenav jump to bottom

telus · Jan 8, 2019 · 845f1e5 · 845f1e5
1 parent dc43d66
commit 845f1e5
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/packages/SideNavigation/SideNavigation.jsx b/packages/SideNavigation/SideNavigation.jsx
@@ -73,11 +73,6 @@ class SideNavigation extends Component {
       sideNavRect.height > containerRect.height
     ) {
       this.setState({ variant: 'top' })
-    } else if (
-      (sideNavRect.bottom > containerRect.bottom || sideNavRect.bottom <= 0) &&
-      sideNavRect.height <= containerRect.height
-    ) {
-      this.setState({ variant: 'bottom' })
     } else if (
       this.checkOverflow(this._sideNav) &&
       sideNavRect.bottom <= containerRect.bottom &&
@@ -94,6 +89,11 @@ class SideNavigation extends Component {
       sideNavRect.height < containerRect.height
     ) {
       this.setState({ variant: 'fixed' })
+    } else if (
+      (sideNavRect.bottom > containerRect.bottom || sideNavRect.bottom <= 0) &&
+      sideNavRect.height <= containerRect.height
+    ) {
+      this.setState({ variant: 'bottom' })
     }
   }