-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Make <amp-sidebar> mask element compatible with Story #24827
Copy link
Copy link
Closed
Labels
Component: amp-sidebarCustomer: DeveloperP2: SoonStaleInactive for one year or moreInactive for one year or moreType: BugType: DevXissues impacting developer experienceissues impacting developer experienceWG: components
Description
Currently, amp-sidebar appends its mask directly under body; this does not work for AMP Story since sidebar is a child of <amp-story> and the default mask would appear above sidebar.
Investigate a way to make the mask element work in this context. We should also ensure that sidebar swipe-to-dismiss works when swiping from the mask inside an AMP story.
/cc @newmuis @gmajoulet
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Component: amp-sidebarCustomer: DeveloperP2: SoonStaleInactive for one year or moreInactive for one year or moreType: BugType: DevXissues impacting developer experienceissues impacting developer experienceWG: components