feat: open "Application Feedback" as regular link

Author: maxceem

src/components/MenuItem/MenuItem.jsx

@@ -23,38 +23,54 @@ const MenuItem = ({
   match,
   wrapperClass,
   toolTipText,
+  enforceA,
+  openNewTab,
 }) => {
   const matchedPath = match && match.path
   const isChildActive = children && some(children, c => c.to === matchedPath)
 
+  const linkContent = (
+    <span styleName="content">
+      <span styleName="left">
+        {!!Icon && (
+          <Icon className={cn(styles.icon, styles[iconClassName])} />
+        )}
+        {label}
+      </span>
+      <span styleName="right">
+        {!!count && <NotificationBadge count={count} text={toolTipText} />}
+      </span>
+    </span>
+  )
+
   return (
     <li className={
       cn(wrapperClass,
         styles.menuItem,
         { [styles.open]: isAccordionOpen, [styles.withChildren]: children })}
     >
-      {!children && (
+      {!children && !enforceA && (
         <NavLink
           to={to}
           className={styles.navItem}
           activeClassName={styles.active}
           exact={exact}
           isActive={isActive}
         >
-          <span styleName="content">
-            <span styleName="left">
-              {!!Icon && (
-                <Icon className={cn(styles.icon, styles[iconClassName])} />
-              )}
-              {label}
-            </span>
-            <span styleName="right">
-              {!!count && <NotificationBadge count={count} text={toolTipText} />}
-            </span>
-          </span>
+          {linkContent}
         </NavLink>
       )}
 
+      {!children && enforceA && (
+        <a
+          href={to}
+          className={styles.navItem}
+          target={openNewTab ? '_blank' : ''}
+        >
+          {linkContent}
+        </a>
+      )}
+
       {children && (
         <div>
           <div
@@ -91,7 +107,9 @@ const MenuItem = ({
 }
 
 MenuItem.defaultProps = {
-  exact: true
+  exact: true,
+  enforceA: false,
+  openNewTab: false,
 }
 
 MenuItem.propTypes = {
@@ -107,6 +125,15 @@ MenuItem.propTypes = {
   onAccordionToggle: PT.func,
   wrapperClass: PT.string,
   toolTipText: PT.string,
+  /**
+   * Render as regular link `<a>` instead of React Router `<NavLink>`
+   */
+  enforceA: PT.bool,
+  /**
+   * Open link in the new tab.
+   * Only works if `enforceA` is `true`.
+   */
+  openNewTab: PT.bool,
 }
 
 export default withRouter(MenuItem)

src/components/UserSidebar/UserSidebar.jsx

@@ -65,7 +65,9 @@ const navLinks = [{
   iconClassName: 'fill',
 }, {
   label: 'GIVE APPLICATION FEEDBACK',
-  to: '/',
+  to: 'https://forms.gle/hwRWYB3cGQnKVpjk7',
+  enforceA: true,
+  openNewTab: true,
   Icon: FAQIcon,
   iconClassName: 'fill',
 }]

src/projects/detail/containers/ProjectInfoContainer.js

@@ -594,7 +594,9 @@ class ProjectInfoContainer extends React.Component {
               navLinks={[
                 {
                   label: 'GIVE APPLICATION FEEDBACK',
-                  to: '/',
+                  to: 'https://forms.gle/hwRWYB3cGQnKVpjk7',
+                  enforceA: true,
+                  openNewTab: true,
                   Icon: FAQIcon,
                   iconClassName: 'fill',
                 }