Desktop Expanded menu not Working on the Twenty Twenty Theme

[fellyph]

Bug Description

The core theme Twenty Twenty, has different Menu Locations and one option is the Desktop Expended Menu:

When this option is selected only appears in the non-AMP version:

non-AMP version:

AMP version:

Expected Behaviour

The AMP version should display the expanded menu option as the non-AMP version

Screenshots

No response

PHP Version

8.0

Plugin Version

2.2.4

AMP plugin template mode

Standard, Transitional

WordPress Version

5.9.3

Site Health

No response

Gutenberg Version

No response

OS(s) Affected

No response

Browser(s) Affected

No response

Device(s) Affected

No response

Acceptance Criteria

No response

Implementation Brief

No response

QA Testing Instructions

No response

Demo

No response

Changelog Entry

No response

[westonruter]

What is the “AMP sidebar menu” theme location there? Where is that coming from?

[westonruter]

We should check to see if this is a recent regression.

[fellyph]

The "AMP Sidebar menu" is from the James's sidebar menu plugin

[dhaval-parekh]

After checking with a different version found that this bug introduces in AMP version 2.2.2 ( because of this change )

Before the change it was converting .no-js .hide-no-js selector into :root:not(#_):not(#_):not(#_):not(#_):not(#_):not(#_):not(#_) .no-js .hide-no-js which was not applying to expanded menu in twenty-twenty theme, But after changes it convert into .no-js .hide-no-js:not(#_#_#_#_#_#_#_) which is apply to expanded menu. ( Note: menu have header-toggles hide-no-js classes )

[pooja-muchandikar]

QA Passed ✅

Cross checked the issue with the test environment mentioned in the issue description and the fix added to the issue is working as expected.

Screen.Recording.2022-06-06.at.8.39.24.PM.mov