Unverified Commit 0bc2ddcf authored by Lauri Timmanee's avatar Lauri Timmanee
Browse files

Issue #3319325 by mherchel, nod_, alexpott: Olivero: Mobile menu does not work... 

Issue #3319325 by mherchel, nod_, alexpott: Olivero: Mobile menu does not work when authenticated and BigPipe enabled(D10 only)

(cherry picked from commit 0b030534)
parent 733663d8

core/tests/Drupal/Nightwatch/Tests/Olivero/oliveroMobileMenuTest.js

+9 −0
Original line number Diff line number Diff line
@@ -172,4 +172,13 @@ module.exports = { 
      .click('[href="#footer"]')

      .waitForElementNotVisible(headerNavSelector);

  },

  'Verify mobile menu works when Big Pipe when authenticated': (browser) => {

    browser.drupalInstallModule('big_pipe').drupalLoginAsAdmin(() => {

      browser

        .drupalRelativeURL('/')

        .assert.not.visible(headerNavSelector)

        .click(mobileNavButtonSelector)

        .waitForElementVisible(headerNavSelector);

    });

  },

};

core/themes/olivero/js/navigation.js

+1 −1
Original line number Diff line number Diff line
@@ -147,7 +147,7 @@ 
        const navButton = context.querySelector(

          '[data-drupal-selector="mobile-nav-button"]',

        );

        const body = context.querySelector('body');

        const body = document.body;

        const overlay = context.querySelector(

          '[data-drupal-selector="header-nav-overlay"]',

        );