fix: now correctly removing activeClass on update

CloudPower97 · Dec 3, 2018 · 10ae785 · 10ae785
1 parent c3aeeb3
commit 10ae785
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Scrollspy.js b/src/Scrollspy.js
@@ -97,8 +97,10 @@ export default class Scrollspy extends Component {
     const { navItems } = this.state
     const { activeClass } = this.props
 
-    const active = navItems.find(navItem => navItem.classList.contains(activeClass))
-    active && active.classList.remove(activeClass)
+    if (navItems) {
+      const active = navItems.find(navItem => navItem.classList.contains(activeClass))
+      active && active.classList.remove(activeClass)
+    }
   }
 
   render() {