fix: Refactor SideNavigation to remove React-Router dependency (#372)

* Save state

* saving state

* Adding interactivity and active item

* Updating Unit Tests

* Downgrading React-Router-* to devDependencies

* Add Prop Descriptons

* Normalized SideNavGroup into SideNavList & renamed SIdeNavSubItems to SideNavSubList

* Removing SubList and Refactoring in SideNavList

* Remove expanded tracking from SideNav and pushing into SideNavListItem

* Linting fixes

* Adding focus-trap after removing it

* Package-lock was not in sync

* PR feedback

* Updating children prop type

* Fixing issue where parent items were being selectd. Also updated documentation

* Correcting testing issue

* Updates to prop descriptions

Author: jeffredodd

package-lock.json

@@ -45,9 +45,7 @@
   "dependencies": {
     "classnames": "^2.2.6",
     "fiori-fundamentals": "^1.4.0",
-    "focus-trap-react": "^6.0.0",
-    "react-router": "^4.3.1",
-    "react-router-dom": "^4.3.1"
+    "focus-trap-react": "^6.0.0"
   },
   "devDependencies": {
     "@babel/cli": "^7.1.5",
@@ -114,6 +112,8 @@
     "react-dev-utils": "^6.1.1",
     "react-dom": "^16.6.3",
     "react-markdown": "^4.0.6",
+    "react-router": "^4.3.1",
+    "react-router-dom": "^4.3.1",
     "react-syntax-highlighter": "^9.0.1",
     "react-test-renderer": "^16.6.3",
     "resolve": "1.8.1",