-
Notifications
You must be signed in to change notification settings - Fork 383
Closed
Labels
ChangeloggedWhether the issue/PR has been added to release notes.Whether the issue/PR has been added to release notes.EnhancementNew feature or improvement of an existing oneNew feature or improvement of an existing oneGroomedIntegration: WP CoreP1Medium priorityMedium priorityRMEReader Mode ExpansionReader Mode ExpansionWS:UXWork stream for UX/Front-endWork stream for UX/Front-end
Milestone
Description
Feature description
When AMP compatibility was added for the Twenty Nineteen theme, we tried our best to implement support for the mobile menu (or rather, to use CSS only to implement the feature: WordPress/twentynineteen#337):
However, we weren't able to implement this without JS (see touch-keyboard-navigation.js), so we had to ship the no-JS fallback experience for AMP (#1587).
However, there is now an AMP component dedicated for this purpose: amp-nested-menu. We should now leverage it in the core theme sanitizer.
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation brief
QA testing instructions
Demo
Changelog entry
jwold, ernee and swissspidy
Metadata
Metadata
Assignees
Labels
ChangeloggedWhether the issue/PR has been added to release notes.Whether the issue/PR has been added to release notes.EnhancementNew feature or improvement of an existing oneNew feature or improvement of an existing oneGroomedIntegration: WP CoreP1Medium priorityMedium priorityRMEReader Mode ExpansionReader Mode ExpansionWS:UXWork stream for UX/Front-endWork stream for UX/Front-end
