Unified 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.
Use n/p to move between diff chunks;
N/P to move between comments.
« includes/abb-notification.tmpl ('K')
|
« locales/en/abb-notification.json ('k')
|
static/css/main.css » ('j')
|
static/css/main.css » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: static/css/index.css |
| =================================================================== |
| --- a/static/css/index.css |
| +++ b/static/css/index.css |
| @@ -58,22 +58,16 @@ |
| #content.ua-android #abb-banner, |
| #content.ua-ios #abb-banner, |
| #content.ua-android #abb-android-download, |
| #content.ua-ios #abb-ios-download |
| { |
| display: block; |
| } |
| -#content.ua-android #adblock-browser-notification, |
| -#content.ua-ios #adblock-browser-notification |
| -{ |
| - display: none; |
| -} |
| - |
| #main, #video-container |
| { |
| display: inline-block; |
| vertical-align: top; /* Fix for https://bugzil.la/1320660 */ |
| } |
| #video-container, #maxthon-instruction, #edge-teaser |
| { |
| @@ -684,8 +678,65 @@ |
| background-color: #fff; |
| border: 2px solid #a1a1a1; |
| } |
| #ready-for-windows-notice |
| { |
| margin-bottom: 30px; |
| } |
| + |
| +/****************************************************************************** |
| + * #abb-notification |
| + *****************************************************************************/ |
| + |
| +#abb-notification |
| +{ |
| + background-color: #ececec; |
| + background-image: url(../img/abb-notification-bg-mobile.png); |
| + background-position: center; |
| + background-repeat: no-repeat; |
| +} |
| + |
| +@media (min-width: 768px) /* tablet-breakpoint */ |
| +{ |
| + #abb-notification |
| + { |
| + background-image: url(../img/abb-notification-bg-tablet.png); |
| + } |
| +} |
| + |
| +@media (min-width: 992px) /* desktop-breakpoint */ |
| +{ |
| + #abb-notification |
| + { |
| + 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.
|
| + } |
| +} |
| + |
| +#abb-notification h2, |
| +#abb-notification h3 |
| +{ |
| + 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.
|
| +} |
| + |
| +#abb-notification a |
| +{ |
| + color: inherit; |
| +} |
| + |
| +.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.
|
| +{ |
| + max-width: 10em; |
| +} |
| + |
| +#abb-app-store-badges img |
| +{ |
| + width: 100%; |
| + 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
|
| +} |
| + |
| +#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.
|
| +#abb-app-store-badges |
| +{ |
| + padding-left: 2em; |
| + padding-right: 2em; |
| +} |
« includes/abb-notification.tmpl ('K')
|
« locales/en/abb-notification.json ('k')
|
static/css/main.css » ('j')
|
static/css/main.css » ('J')
