Merge pull request #2352 from topcoder-platform/legacy-tco

Fix menus bug for root paths

Author: sushilshinde

src/shared/components/Contentful/Menu/Menu.jsx

@@ -3,6 +3,7 @@
  *
  */
 /* global window */
+import _ from 'lodash';
 import React from 'react';
 import PT from 'prop-types';
 import Dropdown from 'components/tc-communities/Dropdown';
@@ -17,7 +18,7 @@ export default function Menu(props) {
   } = props;
 
   if (isomorphy.isClientSide()) {
-    if (baseUrl === parentBaseUrl && baseUrl !== window.location.pathname) {
+    if (baseUrl && baseUrl === parentBaseUrl && baseUrl !== _.trimEnd(window.location.pathname, '/')) {
       return null;
     }
   } else {

src/shared/components/Contentful/Route.jsx

@@ -58,6 +58,7 @@ function ChildRoutesLoader(props) {
                         preview={preview}
                         spaceName={spaceName}
                         environment={environment}
+                        baseUrl={url}
                       />
                     ) : <Error404 />
                 }

src/shared/utils/contentful.js

@@ -64,8 +64,7 @@ export function isActive(baseUrl, item, caller) {
   }
 
   // handles the special case when url === `/`
-  if (caller === 'menuItem' && item
-  && (item.fields.url === '/' || !item.fields.url)
+  if (caller === 'menuItem'
   && to === baseUrl
   && (location !== to && location !== `${to}/`)) {
     return false;