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

Side by Side Diff: static/css/index.css

Issue 29690912: Fixes #9 - Update cross-promotional section (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Created Feb. 6, 2018, 4:24 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 #content 1 #content
2 { 2 {
3 padding-top: 5em; 3 padding-top: 5em;
4 } 4 }
5 5
6 #key-facts h1 6 #key-facts h1
7 { 7 {
8 font-size: 16px; 8 font-size: 16px;
9 } 9 }
10 10
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 } 56 }
57 57
58 #content.ua-android #abb-banner, 58 #content.ua-android #abb-banner,
59 #content.ua-ios #abb-banner, 59 #content.ua-ios #abb-banner,
60 #content.ua-android #abb-android-download, 60 #content.ua-android #abb-android-download,
61 #content.ua-ios #abb-ios-download 61 #content.ua-ios #abb-ios-download
62 { 62 {
63 display: block; 63 display: block;
64 } 64 }
65 65
66 #content.ua-android #adblock-browser-notification,
67 #content.ua-ios #adblock-browser-notification
68 {
69 display: none;
70 }
71
72 #main, #video-container 66 #main, #video-container
73 { 67 {
74 display: inline-block; 68 display: inline-block;
75 vertical-align: top; /* Fix for https://bugzil.la/1320660 */ 69 vertical-align: top; /* Fix for https://bugzil.la/1320660 */
76 } 70 }
77 71
78 #video-container, #maxthon-instruction, #edge-teaser 72 #video-container, #maxthon-instruction, #edge-teaser
79 { 73 {
80 background: #ffffff; 74 background: #ffffff;
81 padding: 10px; 75 padding: 10px;
(...skipping 600 matching lines...) Expand 10 before | Expand all | Expand 10 after
682 line-height: 1.4; 676 line-height: 1.4;
683 color: #000; 677 color: #000;
684 background-color: #fff; 678 background-color: #fff;
685 border: 2px solid #a1a1a1; 679 border: 2px solid #a1a1a1;
686 } 680 }
687 681
688 #ready-for-windows-notice 682 #ready-for-windows-notice
689 { 683 {
690 margin-bottom: 30px; 684 margin-bottom: 30px;
691 } 685 }
686
687 /******************************************************************************
688 * #abb-notification
689 *****************************************************************************/
690
691 #abb-notification
692 {
693 background-color: #ececec;
694 background-image: url(../img/abb-notification-bg-mobile.png);
695 background-position: center;
696 background-repeat: no-repeat;
697 }
698
699 @media (min-width: 768px) /* tablet-breakpoint */
700 {
701 #abb-notification
702 {
703 background-image: url(../img/abb-notification-bg-tablet.png);
704 }
705 }
706
707 @media (min-width: 992px) /* desktop-breakpoint */
708 {
709 #abb-notification
710 {
711 background-image: url(../img/abb-notification-bg-desktop.png);
juliandoucette 2018/02/20 09:23:33 NIT: I think that we should request that this back
ire 2018/02/20 11:16:47 I like this idea but according to the prototype, t
juliandoucette 2018/02/20 14:06:18 Acknowledged. I meant left/right not top/bottom.
712 }
713 }
714
715 #abb-notification h2,
716 #abb-notification h3
717 {
718 text-transform: uppercase;
ire 2018/02/06 16:31:14 All of the headings should be uppercase, but I thi
juliandoucette 2018/02/20 09:23:34 Acknowledged.
719 }
720
721 #abb-notification a
722 {
723 color: inherit;
724 }
725
726 .abb-logo
juliandoucette 2018/02/20 09:23:34 NIT: Why classname? I don't think that this applie
ire 2018/02/20 11:16:47 Agreed. Done.
727 {
728 max-width: 10em;
729 }
730
731 #abb-app-store-badges img
732 {
733 width: 100%;
734 max-width: 200px;
juliandoucette 2018/02/20 09:23:33 Why max width instead of width here?
ire 2018/02/20 11:16:47 My intention was to allow more fluidity, but in pr
735 }
736
737 #abb-notification figure,
juliandoucette 2018/02/20 09:23:34 I think that 2em is too much on small phones. Perh
ire 2018/02/20 11:16:47 Done.
738 #abb-app-store-badges
739 {
740 padding-left: 2em;
741 padding-right: 2em;
742 }
OLDNEW

Powered by Google App Engine
This is Rietveld