Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: pages/test.html

Issue 29703633: Noissue - Abstracted embedded video for use in blog (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Addressed #7, moved video* to main*, and refactored Video class Created Feb. 23, 2018, 1:30 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pages/test.html
===================================================================
new file mode 100644
--- /dev/null
+++ b/pages/test.html
@@ -0,0 +1,44 @@
+title=Video test
+
+<head>
+ <link rel="stylesheet" href="/css/video.css" />
+ <script src="/js/video.js"></script>
+</head>
+
+<div class="video-parent contains-disclaimer">
+ <a
+ class="video-link"
+ target="_blank"
+ href="https://www.youtube-nocookie.com/embed/pVYtzF5SemU?html5=1&amp;autohide=1&amp;enablejsapi=1&amp;controls=2&amp;fs=1&amp;modestbranding=1&amp;rel=0&amp;showinfo=0&amp;theme=light&amp;autoplay=1">
+ <img
+ class="video-thumbnail"
+ src="/img/video-thumbnail.jpg"
+ alt="{{ video-alt-text Introduction to Adblock Plus video thumbnail }}">
+ <img
+ class="video-play"
+ src="/img/video-external.png"
+ alt="{{ video-play-alt-text Play Video }}">
+ </a>
+ <p class="video-disclaimer">
+ {{ video-disclaimer Click to play this video. Please note that this video is hosted by YouTube. When this video is played, some personal data is transferred to YouTube. For more information, please review YouTube's <a href='https://www.google.com/intl/en/policies/privacy/' target='_blank'>privacy policy</a>. }}
+ </p>
+</div>
+
+<div class="video-parent contains-disclaimer" style="max-width: 520px; margin-top: 1em;">
+ <a
+ class="video-link"
+ target="_blank"
+ href="https://www.youtube-nocookie.com/embed/pVYtzF5SemU?html5=1&amp;autohide=1&amp;enablejsapi=1&amp;controls=2&amp;fs=1&amp;modestbranding=1&amp;rel=0&amp;showinfo=0&amp;theme=light&amp;autoplay=1">
+ <img
+ class="video-thumbnail"
+ src="/img/video-thumbnail.jpg"
+ alt="{{ video-alt-text Introduction to Adblock Plus video thumbnail }}">
+ <img
+ class="video-play"
+ src="/img/video-external.png"
+ alt="{{ video-play-alt-text Play Video }}">
+ </a>
+ <p class="video-disclaimer">
+ {{ video-disclaimer Click to play this video. Please note that this video is hosted by YouTube. When this video is played, some personal data is transferred to YouTube. For more information, please review YouTube's <a href='https://www.google.com/intl/en/policies/privacy/' target='_blank'>privacy policy</a>. }}
+ </p>
+</div>

Powered by Google App Engine
This is Rietveld