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

Unified Diff: static/css/index.css

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: static/css/index.css
===================================================================
--- a/static/css/index.css
+++ b/static/css/index.css
@@ -64,109 +64,42 @@
}
#content.ua-android #adblock-browser-notification,
#content.ua-ios #adblock-browser-notification
{
display: none;
}
-#main, #video-container
+#main, #home-video
{
display: inline-block;
vertical-align: top; /* Fix for https://bugzil.la/1320660 */
}
-#video-container, #maxthon-instruction, #edge-teaser
+#maxthon-instruction, #edge-teaser
{
background: #ffffff;
padding: 10px;
border: 1px solid #c9c9c9;
box-shadow: 1px 1px 0 0 #dbdbdb;
}
#maxthon-instruction,
-html[lang^="zh_"] #content.maxthon #video,
+html[lang^="zh_"] #content.maxthon #home-video,
#content.maxthon #feature-free,
#content.maxthon #terms-message,
#feature-maxthon
{
display: none;
}
-#video-container
-{
- position: relative;
- box-sizing: border-box;
- width: 100%;
- max-width: 520px;
-}
-
-#video
-{
- display: block;
- position: relative;
- width: 100%;
-}
-
-#video-thumbnail
-{
- width: 100%;
- transition: opacity 0.8s;
-}
-
-@media(min-width: 768px)
-{
- #video-thumbnail
- {
- height: 275px;
- }
-}
-
-.no-js #video-thumbnail,
-.show-disclaimer #video-thumbnail
+#home-video
{
- opacity: 0.2;
-}
-
-#video-disclaimer
-{
- visibility: hidden;
- opacity: 0;
- position: absolute;
- bottom: 0;
- left: 0;
- margin: 20px;
- background-color: #fff;
- font-size: 0.9em;
- text-align: center;
- transition: opacity 0.8s;
-}
-
-.no-js #video-disclaimer,
-.show-disclaimer #video-disclaimer
-{
- visibility: visible;
- opacity: 1;
-}
-
-#video-play
-{
- position: absolute;
- top: 50%;
- left: 50%;
- width: 80px;
- margin-top: -40px;
- margin-left: -40px;
-}
-
-.no-js #video-play,
-.show-disclaimer #video-play
-{
- margin-top: -60px;
+ max-width: 520px;
}
html[lang^="zh_"] #content.maxthon #maxthon-instruction
{
display: inline-block;
padding: 0px;
}
@@ -516,17 +449,17 @@
#content h3.subscribe-header
{
font-size: 18px;
margin-bottom: 0px;
}
#other-platform > div,
-#content.edge #video,
+#content.edge #home-video,
#edge-teaser
{
display: none;
}
.warning-platform,
.disclaimer,
#maxthon-notification

Powered by Google App Engine
This is Rietveld