feat(utility): addressed the review comments

Author: Mahesh Kothur

packages/react/src/patterns/sub-patterns/TableOfContents/TOCDesktop.js

@@ -66,7 +66,7 @@ const TOCDesktop = ({ menuItems, selectedId, updateState }) => {
     const title = filteredItems[0].title;
     updateState(id, title);
     const selector = `a[name="${id}"]`;
-    smoothScroll({ e: null, selector });
+    smoothScroll({ selector });
   };
 
   /**

packages/react/src/patterns/sub-patterns/TableOfContents/TOCMobile.js

@@ -72,7 +72,7 @@ const TOCMobile = ({ menuItems, selectedId, menuLabel, updateState }) => {
     const title = filteredItems[0].title;
     updateState(id, title);
     const selector = `a[name="${id}"]`;
-    smoothScroll({ e: null, selector });
+    smoothScroll({ selector });
   };
 
   /**

packages/utilities/src/utilities/smoothScroll/smoothScroll.js

@@ -20,18 +20,19 @@
  * @param {object} { e: Event, Selector: QuerySelector for the target element }
  */
 const smoothScroll = ({ e, selector }) => {
-  if (e !== null) {
+  let getSelector;
+  if (e) {
     e.preventDefault();
-    const id = e.currentTarget.getAttribute('href');
-    document.querySelector(id).scrollIntoView({
-      behavior: 'smooth',
-      block: 'start',
-    });
-  } else if (selector !== null) {
-    document.querySelector(selector).scrollIntoView({
-      behavior: 'smooth',
-      block: 'start',
-    });
+    getSelector = e.currentTarget.getAttribute('href');
+  } else if (selector) {
+    getSelector = selector;
+  } else {
+    return null;
   }
+  document.querySelector(getSelector).scrollIntoView({
+    behavior: 'smooth',
+    block: 'start',
+  });
 };
+
 export default smoothScroll;