BACKLOG-21267: Display popup only in Page Composer view

src/javascript/JContent/ContentRoute/ContentPath/ContentPath.container.jsx

@@ -7,6 +7,7 @@ import {GetContentPath} from './ContentPath.gql-queries';
 import ContentPath from './ContentPath';
 import {ContentPathDialog} from './ContentPathDialog';
 import PropTypes from 'prop-types';
+import JContentConstants from '~/JContent/JContent.constants';
 
 function findLastIndex(array, callback) {
     let lastIndex = -1;
@@ -43,14 +44,14 @@ const ContentPathContainer = ({setPathAction, selector}) => {
     const dispatch = useDispatch();
     const [currentItem, setCurrentItem] = useState(null);
 
-    const {mode, language, path} = useSelector(selector, shallowEqual);
+    const {mode, language, path, viewMode} = useSelector(selector, shallowEqual);
 
     const {data, error} = useQuery(GetContentPath, {
         variables: {path, language}
     });
 
     const handleNavigation = item => {
-        if (item.primaryNodeType?.name === 'jnt:contentList') {
+        if (item.primaryNodeType?.name === 'jnt:contentList' && mode === JContentConstants.mode.PAGES && viewMode === JContentConstants.tableView.viewMode.PAGE_COMPOSER) {
             setCurrentItem(item);
         } else {
             dispatch(setPathAction(mode, item.path));
@@ -88,6 +89,7 @@ ContentPathContainer.propTypes = {
 ContentPathContainer.defaultProps = {
     selector: state => ({
         mode: state.jcontent.mode,
+        viewMode: state.jcontent.tableView.viewMode,
         path: state.jcontent.path,
         language: state.language
     }),

src/javascript/JContent/ContentRoute/ContentPath/ContentPath.container.test.jsx

@@ -34,7 +34,10 @@ describe('ContentPathContainer', () => {
         useSelector.mockImplementation(callback => callback({
             language: 'fr',
             jcontent: {
-                path: '/x/y/z'
+                path: '/x/y/z',
+                tableView: {
+                    viewMode: 'bar'
+                }
             }
         }));
 
@@ -64,6 +67,14 @@ describe('ContentPathContainer', () => {
             }
         }));
 
+        useSelector.mockImplementation(callback => callback({
+            jcontent: {
+                tableView: {
+                    viewMode: 'bar'
+                }
+            }
+        }));
+
         const wrapper = shallow(<ContentPathContainer/>).find('ContentPath');
         expect(wrapper.type()).toBe(ContentPath);
         expect(wrapper.prop('items')).toEqual(ancestors);
@@ -77,7 +88,10 @@ describe('ContentPathContainer', () => {
 
         useSelector.mockImplementation(callback => callback({
             jcontent: {
-                mode: 'foo'
+                mode: 'foo',
+                tableView: {
+                    viewMode: 'bar'
+                }
             }
         }));
 

tests/cypress/e2e/jcontent/breadcrumbs.cy.ts

@@ -10,23 +10,35 @@ describe('Breadcrumb navigation test', () => {
         cy.loginAndStoreSession();
     });
 
-    it('Handles area and parent page navigation correctly', () => {
-        let jcontent = JContent.visit('digitall', 'en', 'pages/home/newsroom/news-entry/article/all-organic-foods-network-gains');
+    it('Display popup when navigating to list and render list view when selected', () => {
+        const jcontent = JContent.visit('digitall', 'en', 'pages/home/newsroom/news-entry/article/all-organic-foods-network-gains');
         jcontent.switchToPageComposer();
         Breadcrumb.findByContent('article').click();
         cy.get('button[data-cm-role="breadcrumb-view-list"]').should('be.visible').click();
         cy.get('.moonstone-chip').find('span').contains('Content List').should('be.visible');
         cy.get('h1').contains('article');
+    });
 
-        jcontent = JContent.visit('digitall', 'en', 'pages/home/newsroom/news-entry/article/all-organic-foods-network-gains');
+    it('Display popup when navigating to list and render parent page when selected', () => {
+        const jcontent = JContent.visit('digitall', 'en', 'pages/home/newsroom/news-entry/article/all-organic-foods-network-gains');
         jcontent.switchToPageComposer();
         Breadcrumb.findByContent('article').click();
         cy.get('button[data-cm-role="breadcrumb-view-parent"]').should('be.visible').click();
         cy.get('.moonstone-chip').find('span').contains('Page').should('be.visible');
         cy.get('h1').contains('News Entry');
+    });
 
+    it('Do not display popup when navigating to page', () => {
+        const jcontent = JContent.visit('digitall', 'en', 'pages/home/newsroom/news-entry');
+        jcontent.switchToPageComposer();
         Breadcrumb.findByContent('Newsroom').click();
         cy.get('.moonstone-chip').find('span').contains('Page').should('be.visible');
         cy.get('h1').contains('Newsroom');
     });
+
+    it('Do not display popup when navigating to list in list view', () => {
+        JContent.visit('digitall', 'en', 'pages/home/investors/events/Events/ceos-of-the-digital-roundtable/relatedPeople');
+        Breadcrumb.findByContent('CEOs of The Digital Roundtable').click();
+        cy.get('.moonstone-chip').find('span').contains('Event').should('be.visible');
+    });
 });