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

Unified Diff: includes/index.tmpl

Issue 29727563: Fixes #35 - Progressively enhance install button with appropriate links and text (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Addressed comments #12 Created April 9, 2018, 9:22 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 | « includes/index.html ('k') | static/css/index.css » ('j') | static/js/index.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/index.tmpl
===================================================================
new file mode 100644
--- /dev/null
+++ b/includes/index.tmpl
@@ -0,0 +1,30 @@
+title=Adblock Plus | The world's # 1 free ad blocker
+description=Adblock Plus, the most popular ad blocker on Firefox, Chrome, Safari, Android and iOS. Block pop-ups and annoying ads on websites like Facebook and YouTube.
+
+<head>
+ <link rel="stylesheet" href="/css/index.css">
+ <link rel="chrome-webstore-item" href="https://chrome.google.com/webstore/detail/cfhdojbkjhnklbpkdaibdccddilifddb">
+</head>
+
+<? include hero-download ?>
+<? include as-mentioned ?>
+<? include features ?>
+<? include abb-cross-promotion ?>
+<? include footnote ?>
+
+<script id="hero-download-button-template" type="text/template">
juliandoucette 2018/04/09 13:01:02 NIT: I'd prefer to output translations into a sing
ire 2018/04/12 08:05:46 Ack, I think we can handle this separately then.
+ {% if locale in ["en", "de", "fr"] %}
+ {{ "Agree and Install for <fix>browser</fix>" | translate("agree-and-install-for-browser", "button label") }}
+ {% else %}
+ {{ "Install for <fix>browser</fix>" | translate("install-for-browser", "button label") }}
+ {% endif %}
+</script>
+
+{% for browser in browsers %}
+ <script id="{{ browser.id }}-name-translated" type="text/template">
+ {{ browser.name | translate(browser.id + "-name") }}
+ </script>
+{% endfor %}
+
+<script src="/js/vendor/bowser.js"></script>
+<script src="/js/index.js"></script>
« no previous file with comments | « includes/index.html ('k') | static/css/index.css » ('j') | static/js/index.js » ('J')

Powered by Google App Engine
This is Rietveld