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

Delta Between Two Patch Sets: templates/article.tmpl

Issue 29559620: Issue 5692 - Create Browser Selector with Browser Detection Component for help.eyeo.com (Closed) Base URL: https://hg.adblockplus.org/help.eyeo.com
Left Patch Set: Extend CustomSelect, address NITs Created Oct. 27, 2017, 10:22 a.m.
Right Patch Set: Re-add browser-select include, add scripts block Created Nov. 2, 2017, 2:29 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
LEFTRIGHT
1 {% extends "templates/minimal" %} 1 {% extends "templates/minimal" %}
2 2
3 {% set product = products[product_id] %} 3 {% set product = products[product_id] %}
4 4
5 {% block body %} 5 {% block body %}
6 <nav aria-label="{{ "Breadcrumb" | translate("breadcrumb-label", "Label") }}" cl ass="breadcrumb"> 6 <nav aria-label="{{ "Breadcrumb" | translate("breadcrumb-label", "Label") }}" cl ass="breadcrumb">
7 <ol class="container large-desktop-width clearfix" itemscope itemtype="http:// schema.org/BreadcrumbList"> 7 <ol class="container large-desktop-width clearfix" itemscope itemtype="http:// schema.org/BreadcrumbList">
8 <li itemprop="itemListElement" itemscope itemtype="http://schema.org/ListIte m"> 8 <li itemprop="itemListElement" itemscope itemtype="http://schema.org/ListIte m">
9 <a itemscope itemtype="http://schema.org/Thing" itemprop="item" href="{{ p roduct.slug }}"> 9 <a class="has-pre-icon" itemscope itemtype="http://schema.org/Thing" itemp rop="item" href="{{ product.slug }}">
10 <img class="heading-icon" src="/img/png/logo-{{ product_id }}.png" srcse t="/img/svg/logo-{{ product_id }}.svg 2x" alt="{{ product.full_name+" Logo" | tr anslate(product_id+"-logo-alt", "Image alt text") }}"> 10 <span class="pre-icon" style="{{ get_inline_bg('logo-'+product_id) }}">< /span>
11 <span itemprop="name">{{ product.full_name | translate(product_id+"-name ", "Product name") }}</span> 11 <span itemprop="name">{{ product.full_name | translate(product_id+"-name ", "Product name") }}</span>
12 </a> 12 </a>
13 <meta itemprop="position" content="1" /> 13 <meta itemprop="position" content="1" />
14 </li> 14 </li>
15 <li itemprop="itemListElement" itemscope itemtype="http://schema.org/ListIte m"> 15 <li itemprop="itemListElement" itemscope itemtype="http://schema.org/ListIte m">
16 <a itemscope itemtype="http://schema.org/Thing" itemprop="item" href="{{ p age }}" aria-current="page"> 16 <a itemscope itemtype="http://schema.org/Thing" itemprop="item" href="{{ p age }}" aria-current="page">
17 <span itemprop="name">{{ title | translate(get_page_name(page) + "-title ", "Article title") }}</span> 17 <span itemprop="name">{{ title | translate(get_page_name(page) + "-title ", "Article title") }}</span>
18 </a> 18 </a>
19 <meta itemprop="position" content="2" /> 19 <meta itemprop="position" content="2" />
20 </li> 20 </li>
21 </ol> 21 </ol>
22 </nav> 22 </nav>
23 23
24 <main id="main" class="container clearfix"> 24 <main id="main" class="container clearfix">
25 <article class="article card {{ product_id }}-card section column two-thirds"> 25 <div class="row">
26 <h1 class="article-heading"> 26 <div class="column two-thirds">
27 <img class="heading-icon" src="/img/png/logo-{{ product_id }}.png" srcse t="/img/svg/logo-{{ product_id }}.svg 2x" alt="{{ product.full_name+" Logo" | tr anslate( product_id+"-logo-alt", "Image alt text") }}"> 27 <article class="article card {{ product_id }}-card section">
28 <header class="article-heading">
29 <h1 class="has-pre-icon">
30 <span class="pre-icon" style="{{ get_inline_bg('logo-'+product_id) } }"></span>
31 {{ title | translate( get_page_name(page) + "-title", "Article title ") }}
32 </h1>
33 </header>
28 34
29 {{ title | translate( get_page_name(page) + "-title", "Article title") } } 35 {% if hide_browser_selector is not defined %}
30 </h1> 36 <div class="article-browser-selector">
31 37 <? include browser-select ?>
32 {% if hide_browser_selector is not defined %} 38 </div>
33 <div class="article-browser-selector"> 39 {% endif %}
34 <? include browser-select ?>
35 </div>
36 {% endif %}
37 40
38 <div class="article-body content"> 41 <div class="article-body content">
39 {{ body | safe }} 42 {{ body | safe }}
40 </div> 43 </div>
41 </article> 44 </article>
42 <aside class="section column one-third"> 45 </div>
43 <? include product-topics-accordion ?> 46 <aside class="section column one-third">
44 </aside> 47 <? include product-topics-accordion ?>
48 </aside>
49 </div>
45 </main> 50 </main>
46 51
47 <script id="no-content-for-platform-message" type="text/template"> 52 <script id="no-content-for-platform-message" type="text/template">
48 <p>{{ "Unfortunately, there is no content specific for your current browser. P lease use the dropdown menu above to choose a different browser." | translate("n o-content-for-platform-message", "paragraph") }}</p> 53 <p>{{ "Unfortunately, there is no content specific for your current browser. P lease use the dropdown menu above to choose a different browser." | translate("n o-content-for-platform-message", "paragraph") }}</p>
49 </script> 54 </script>
50
51 <script src="/js/vendor/bowser.js"></script>
52 {% endblock %} 55 {% endblock %}
53 56
54 {% block footer %} 57 {% block footer %}
55 <? include contact ?> 58 <? include contact ?>
56 {% endblock %} 59 {% endblock %}
60
61 {% block scripts %}
62 <script src="/js/vendor/bowser.js"></script>
63 {% endblock %}
LEFTRIGHT

Powered by Google App Engine
This is Rietveld