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

Unified Diff: includes/abb-notification.tmpl

Issue 29690912: Fixes #9 - Update cross-promotional section (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Change background image to solid colour Created Feb. 28, 2018, 8:40 a.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
« no previous file with comments | « no previous file | includes/index.tmpl » ('j') | locales/en/abb-notification.json » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/abb-notification.tmpl
===================================================================
--- a/includes/abb-notification.tmpl
+++ b/includes/abb-notification.tmpl
@@ -1,7 +1,22 @@
-<div id="adblock-browser-notification" class="notice">
- {{get_string("abb_notification", "abb-notification")}}
- <span>
- <img src="/img/abb-logo.png" alt="{{get_string("abb_logo_alt", "abb-notification")}}" />
- <a href="https://adblockbrowser.org">{{get_string("abb_notification_link", "abb-notification")}}</a>
- </span>
-</div>
+<section id="abb-notification" class="section bg-secondary">
juliandoucette 2018/02/28 19:48:21 NIT: This isn't really a "notification" anymore IM
ire 2018/03/01 09:11:10 You already brought this up here - https://coderev
juliandoucette 2018/03/01 09:53:33 Yes, thank you. I'll respond there.
+ <div class="container phablet-width content text-center">
+ <h2 class="h1">{{ get_string("abb_notification", "abb-notification") }}</h2>
+
+ <figure>
+ <img src="/img/abb-logo.png" alt="{{ get_string("abb_logo_alt", "abb-notification") }}" id="abb-logo">
+ <figcaption>
+ <h3>{{ get_string("abb_notification_link", "abb-notification") }}</h3>
+ <p>{{ get_string("abb_notification_description", "abb-notification") }}</p>
+ </figcaption>
+ </figure>
+
+ <div id="abb-app-store-badges" class="row">
+ <a href="https://eyeo.to/adblockbrowser/ios/abp-website" title="{{ get_string("abb_ios_link_title", "abb-notification") }}" class="column one-half">
+ <img src="/img/apple-app-store-badge.png" srcset="/img/apple-app-store-badge.svg 2x" alt="{{ get_string("abb_ios_badge_alt", "abb-notification") }}">
+ </a>
+ <a href="https://eyeo.to/adblockbrowser/android/abp-website" title="{{ get_string("abb_android_link_title", "abb-notification") }}" class="column one-half">
+ <img src="/img/google-play-badge.png" srcset="/img/google-play-badge.svg 2x" alt="{{ get_string("abb_android_badge_alt", "abb-notification") }}">
+ </a>
+ </div>
+ </div>
+</section>
« no previous file with comments | « no previous file | includes/index.tmpl » ('j') | locales/en/abb-notification.json » ('J')

Powered by Google App Engine
This is Rietveld