Skip to content

Use amp-nested-menu to implement mobile menu in Twenty Nineteen #4400

@westonruter

Description

@westonruter

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):

Comps for submenu

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    ChangeloggedWhether the issue/PR has been added to release notes.EnhancementNew feature or improvement of an existing oneGroomedIntegration: WP CoreP1Medium priorityRMEReader Mode ExpansionWS:UXWork stream for UX/Front-end

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions