Unified Diff: pages/download.tmpl
Issue 29722640:
Fixes #22 - Add download page (Closed)
Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Rebase
Created March 21, 2018, 9:17 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« includes/features.html ('K')
|
« includes/features.html ('k')
|
static/css/index.css » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: pages/download.tmpl |
| =================================================================== |
| new file mode 100644 |
| --- /dev/null |
| +++ b/pages/download.tmpl |
| @@ -0,0 +1,96 @@ |
| +title=Download Adblock Plus |
| +description=Get Adblock Plus for free on Firefox, Chrome, Opera, Safari, Android and iOS |
| +template=minimal |
| + |
| +<head> |
| + <style> |
| + header img { |
| + width: 4em; |
| + height: auto; |
| + } |
| + </style> |
| +</head> |
| + |
| + |
| + |
| +{% set supported_devices = [ |
| + { |
| + "id": "desktop", |
| + "icons": ["device-desktop.png", "device-desktop-2x.png"], |
| + "icon_alt": "Computer monitor and mouse icon", |
| + "title": "Block ads on any desktop browser", |
| + "description": "By clicking any of the links below, you agree to our " + "terms" | linkify + "Terms of Use</a>.", |
|
juliandoucette
2018/04/03 13:24:51
NIT/Suggest: <small> (semantic, not size.)
ire
2018/04/03 15:37:37
Ack. Done.
|
| + "platforms": [ |
| + { "name": "Chrome", "url": "https://chrome.google.com/webstore/detail/cfhdojbkjhnklbpkdaibdccddilifddb" }, |
| + { "name": "Firefox", "url": "https://update.adblockplus.org/latest/adblockplusfirefox.xpi" }, |
| + { "name": "Internet Explorer", "url": "https://update.adblockplus.org/latest/adblockplusie.exe" }, |
| + { "name": "Safari", "url": "https://update.adblockplus.org/latest/adblockplussafari.safariextz" }, |
| + { "name": "Edge (beta)", "url": "https://www.microsoft.com/store/p/adblock-plus/9nblggh4r9nz" }, |
| + { "name": "Opera", "url": "https://addons.opera.com/extensions/details/opera-adblock/" }, |
| + { "name": "Maxthon", "url": "http://www.maxthon.com/mx5/features/ad-blocker/" }, |
| + { "name": "Yandex Browser", "url": "https://chrome.google.com/webstore/detail/cfhdojbkjhnklbpkdaibdccddilifddb" } |
| + ] |
| + }, |
| + { |
| + "id": "mobile", |
| + "icons": ["device-mobile.png", "device-mobile.svg"], |
| + "icon_alt": "Mobile icon", |
| + "title": "Block ads in your mobile browser", |
| + "platforms": [ |
| + { "name": "Samsung Internet (Android)", "url": "https://play.google.com/store/apps/details?id=org.adblockplus.adblockplussbrowser&hl" }, |
| + { "name": "Safari (iOS)", "url": "https://itunes.apple.com/app/adblock-plus-abp-remove-ads-browse-faster-without-tracking/id1028871868?mt=8" }, |
| + { "name": "Firefox", "url": "https://addons.mozilla.org/android/addon/adblock-plus/" } |
| + ] |
| + }, |
| + { |
| + "id": "abb", |
| + "icons": ["abb-icon.png", "abb-icon-2x.png"], |
| + "icon_alt": "Adblock Browser logo", |
| + "title": "Block ads in a dedicated ad-blocking browser", |
| + "platforms": [ |
| + { "name": "Adblock Browser (Android)", "url": "https://play.google.com/store/apps/details?id=org.adblockplus.browser" }, |
|
saroyanm
2018/03/23 18:33:39
I noticed that we still not using redirection link
ire
2018/03/26 07:29:07
Ack. Will wait for Jeen's response.
|
| + { "name": "Adblock Browser (iOS)", "url": "https://itunes.apple.com/my/app/adblock-browser/id1015653330?mt=8&ign-itsct=1015653330-1015653330&ign-itscg=0176&ign-mpt=uo%3D4" } |
| + ] |
| + } |
| +] %} |
| + |
| +<header class="bg-primary section content text-center"> |
| + <div class="container"> |
| + <img height="90px" |
| + width="90px" |
| + src="/img/download-icon.png" |
| + srcset="/img/download-icon.svg 2x" |
| + alt="{{ "White down arrow" | translate("download-icon", "Alt text") }}"> |
| + <h1>{{ "Get Adblock Plus products on these devices" | translate("download-heading", "Heading") }}</h1> |
| + <p>{{ "Supported platforms" | translate("download-subheading", "Subheading") }}</p> |
| + </div> |
| +</header> |
| + |
| +<div class="item-group container text-center content"> |
| + <div class="row"> |
| + {% for device in supported_devices %} |
| + <div class="column one-third"> |
|
juliandoucette
2018/04/03 13:24:51
NIT/Suggest: <article>
ire
2018/04/03 15:37:37
I don't think this fits within the definition of a
|
| + <img |
| + height="90px" |
| + width="90px" |
| + src="/img/{{ device.icons[0] }}" |
| + srcset="/img/{{ device.icons[1] }} 2x" |
|
juliandoucette
2018/04/03 13:24:51
NIT: You could use a dict and place device.icons.1
ire
2018/04/03 15:37:37
Done.
|
| + alt="{{ device.icon_alt | translate(device.id + "-icon-alt", "Alt text") }}"> |
|
juliandoucette
2018/04/03 13:24:51
NIT: I usually imagine "id"s to be for machines an
ire
2018/04/03 15:37:37
I look at "id"s simply as "identifiers". I wouldn'
|
| + <h2>{{ device.title | translate(device.id + "-title", "Heading") }}</h2> |
|
juliandoucette
2018/04/03 13:24:50
NIT: It's unnecessary to capitalize string context
ire
2018/04/03 15:37:37
I like it capitalised. It conveys to me that it is
|
| + <div> |
| + {% if device.description %} |
| + <p>{{ device.description | safe | translate(device.id + "-description", "Paragraph") }}</p> |
| + {% endif %} |
| + <ul class="horizontal-list"> |
| + {% for platform in device.platforms %} |
| + <li> |
| + {% set platform_identifier = device.id + "-product-name-" + loop.index | string %} |
| + <a href="{{ platform.url }}">{{ platform.name | translate(platform_identifier, "Product name") }}</a>{{ ", " if loop.index != loop.length }} |
|
juliandoucette
2018/04/03 13:24:51
Cool :+1:
ire
2018/04/03 15:37:37
Thanks :p
|
| + </li> |
| + {% endfor %} |
| + </ul> |
| + </div> |
| + </div> |
| + {% endfor %} |
| + </div> |
| +</div> |
« includes/features.html ('K')
|
« includes/features.html ('k')
|
static/css/index.css » ('j')
|
no next file with comments »
