refactor(sharing): avoid nested template literals

This improves readability. I also rename a variable to avoid duplicate
between global scope and useEffect scope.

1 files changed, 4 insertions, 7 deletions

diff --git a/src/components/Icons/Hamburger/Hamburger.tsx b/src/components/Icons/Hamburger/Hamburger.tsx
index 3b9e609..9b39272 100644
--- a/src/components/Icons/Hamburger/Hamburger.tsx
+++ b/src/components/Icons/Hamburger/Hamburger.tsx
@@ -1,13 +1,10 @@
 import styles from './Hamburger.module.scss';
 
 const HamburgerIcon = ({ isActive }: { isActive: boolean }) => {
-  return (
-    <span
-      className={`${styles.icon}${
-        isActive ? ` ${styles['icon--active']}` : ''
-      }`}
-    ></span>
-  );
+  const withModifier = isActive ? ` ${styles['icon--active']}` : '';
+  const iconClasses = `${styles.icon} ${withModifier}`;
+
+  return <span className={iconClasses}></span>;
 };
 
 export default HamburgerIcon;