Skip to content

Commit

Permalink
🐛 Make amp-consent position: fixed !important (#26819)
Browse files Browse the repository at this point in the history
Since this component is always filled asynchronously, not forcing it fixed causes content shift.
  • Loading branch information
alanorozco committed Mar 29, 2020
1 parent bcde1ec commit 5dc2404
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion extensions/amp-consent/0.1/amp-consent.css
Expand Up @@ -32,7 +32,7 @@

amp-consent {
/* Fixed to make position independent of page other elements. */
position: fixed;
position: fixed !important;
bottom: 0;
left: 0;
overflow: hidden !important;
Expand Down
2 changes: 1 addition & 1 deletion extensions/amp-story/0.1/amp-story.css
Expand Up @@ -24,7 +24,7 @@ amp-story, amp-story-page, amp-story-grid-layer, amp-story-cta-layer {
overflow: hidden !important;
}

amp-consent {
amp-story amp-consent {
position: absolute !important;
top: 0 !important;
left: 0 !important;
Expand Down
2 changes: 1 addition & 1 deletion extensions/amp-story/1.0/amp-story.css
Expand Up @@ -30,7 +30,7 @@ amp-story, amp-story-page, amp-story-grid-layer, amp-story-cta-layer {
-webkit-touch-callout: none !important;
}

amp-consent {
amp-story amp-consent {
position: absolute !important;
top: 0 !important;
left: 0 !important;
Expand Down

0 comments on commit 5dc2404

Please sign in to comment.