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

Delta Between Two Patch Sets: templates/article.tmpl

Issue 29588588: Issue 5848 - Fix vertical alignment on .heading-icon images in help.eyeo.com (Closed) Base URL: https://hg.adblockplus.org/help.eyeo.com
Left Patch Set: Created Oct. 25, 2017, 10:21 a.m.
Right Patch Set: Rewrite .has-pre-icon Created Oct. 30, 2017, 9:44 a.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
« no previous file with change/comment | « static/scss/main.scss ('k') | templates/product-home.tmpl » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 [Breadcrumbs] 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">
8 <li itemprop="itemListElement" itemscope itemtype="http://schema.org/ListIte m">
9 <a class="has-pre-icon" itemscope itemtype="http://schema.org/Thing" itemp rop="item" href="{{ product.slug }}">
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>
12 </a>
13 <meta itemprop="position" content="1" />
14 </li>
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">
17 <span itemprop="name">{{ title | translate(get_page_name(page) + "-title ", "Article title") }}</span>
18 </a>
19 <meta itemprop="position" content="2" />
20 </li>
21 </ol>
22 </nav>
7 23
8 <main id="main" class="container clearfix"> 24 <main id="main" class="container clearfix">
9 <article class="article card {{ product_id }}-card section column two-thirds"> 25 <article class="article card {{ product_id }}-card section column two-thirds">
10 <header class="article-heading"> 26 <header class="article-heading">
11 <h1 class="aside-icon-container"> 27 <h1 class="has-pre-icon">
12 <img src="/img/png/logo-{{ product_id }}.png" srcset="/img/svg/logo-{{ p roduct_id }}.svg 2x" alt="{{ product.full_name+" Logo" | translate( product_id+" -logo-alt", "Image alt text") }}"> 28 <span class="pre-icon" style="{{ get_inline_bg('logo-'+product_id) }}">< /span>
13 29 {{ title | translate( get_page_name(page) + "-title", "Article title") } }
ire 2017/10/30 09:46:41 This looks odd when the article title flows on to
juliandoucette 2017/10/31 14:14:35 Acknowledged.
ire 2017/11/01 07:57:25 https://issues.adblockplus.org/ticket/5963
14 <span>{{ title | translate( get_page_name(page) + "-title", "Article tit le") }}</span>
15 </h1> 30 </h1>
16 </header> 31 </header>
17 32
18 {% if hide_browser_selector is not defined %} 33 {% if hide_browser_selector is not defined %}
19 <div class="article-browser-selector"> 34 <div class="article-browser-selector">
20 [Browser selector] 35 [Browser selector]
21 </div> 36 </div>
22 {% endif %} 37 {% endif %}
23 38
24 <div class="article-body content"> 39 <div class="article-body content">
25 {{ body | safe }} 40 {{ body | safe }}
26 </div> 41 </div>
27 </article> 42 </article>
28 <aside class="section column one-third"> 43 <aside class="section column one-third">
29 <? include product-topics-accordion ?> 44 <? include product-topics-accordion ?>
30 </aside> 45 </aside>
31 </main> 46 </main>
32 47
33 <script id="no-content-for-platform-message" type="text/template"> 48 <script id="no-content-for-platform-message" type="text/template">
34 <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 <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>
35 </script> 50 </script>
36 {% endblock %} 51 {% endblock %}
37 52
38 {% block footer %} 53 {% block footer %}
39 <? include contact ?> 54 <? include contact ?>
40 {% endblock %} 55 {% endblock %}
LEFTRIGHT

Powered by Google App Engine
This is Rietveld