Skip to content

Commit

Permalink
Added description and some css for it in analytics-notification to so…
Browse files Browse the repository at this point in the history
  • Loading branch information
ajitsinghkaler authored and Micajuine Ho committed Dec 9, 2019
1 parent 4cf0ca7 commit b645808
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion examples/analytics-notification.amp.html
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@
padding: 8px;
background: #46b6ac;
}

.analytic-notification-demo-text{
padding:16px;
}

amp-user-notification .btn {
border: none;
Expand Down Expand Up @@ -62,7 +66,12 @@
<script async src="https://cdn.ampproject.org/v0.js"></script>
</head>
<body>

<div class="analytic-notification-demo-text">
<h3>AMP Analytics Notification Demo</h3>
<p>This page uses the <a href="https://amp.dev/documentation/components/amp-user-notification/">amp-user-notification</a> to display a cookie concent banner at the bottom of the page.</p>
<p>When user dismisses the notification, the <a href="https://amp.dev/documentation/components/amp-analytics/">amp-analytics</a> component is used to post a 'cookie-concent-ping' request to an analytics server.</p>
<p>The analytics server doesn't exist so you will see an error in developer console.</p>
</div>
<amp-user-notification
layout=nodisplay
id="amp-user-notification1"
Expand Down

0 comments on commit b645808

Please sign in to comment.