-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Closed
Description
relates to #18828
relates to #19841
cc @zhouyx
After (UI/UX) design review with @spacedino , we came to the following remaning UI / UX things we should address before completing the AMP Consent v2 UI/UX:
- Implement Nicer Example in
diy-consent.html - Fix the Flash of the "teal" background in
diy-consent.html - Compensate for the iOS Safari Nav Bar.
- If easy fix, have the consent bound to the bottom of the nav bar, and raise and lower with the nav bar.
- If not easy, make the height
calc(30vh + /*iOS Bottom Nav Bar Height*/), and document for CMPs - Update
diy-consent.htmlto show this behavior
- Add drop shadow to the loading placeholder bottom footer thing (same shadow from sticky ad)
- Add 8px Rounded corners
Reactions are currently unavailable