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

Unified Diff: 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
Patch Set: Created Oct. 25, 2017, 10:21 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
Index: templates/article.tmpl
===================================================================
--- a/templates/article.tmpl
+++ b/templates/article.tmpl
@@ -2,31 +2,33 @@
{% set product = products[product_id] %}
{% block body %}
[Breadcrumbs]
<main id="main" class="container clearfix">
<article class="article card {{ product_id }}-card section column two-thirds">
- <h1 class="article-heading">
- <img class="heading-icon" src="/img/png/logo-{{ product_id }}.png" srcset="/img/svg/logo-{{ product_id }}.svg 2x" alt="{{ product.full_name+" Logo" | translate( product_id+"-logo-alt", "Image alt text") }}">
+ <header class="article-heading">
+ <h1 class="aside-icon-container">
+ <img src="/img/png/logo-{{ product_id }}.png" srcset="/img/svg/logo-{{ product_id }}.svg 2x" alt="{{ product.full_name+" Logo" | translate( product_id+"-logo-alt", "Image alt text") }}">
- {{ title | translate( get_page_name(page) + "-title", "Article title") }}
+ <span>{{ title | translate( get_page_name(page) + "-title", "Article title") }}</span>
</h1>
-
- {% if hide_browser_selector is not defined %}
- <div class="article-browser-selector">
- [Browser selector]
- </div>
- {% endif %}
+ </header>
- <div class="article-body content">
- {{ body | safe }}
- </div>
+ {% if hide_browser_selector is not defined %}
+ <div class="article-browser-selector">
+ [Browser selector]
+ </div>
+ {% endif %}
+
+ <div class="article-body content">
+ {{ body | safe }}
+ </div>
</article>
<aside class="section column one-third">
<? include product-topics-accordion ?>
</aside>
</main>
<script id="no-content-for-platform-message" type="text/template">
<p>{{ "Unfortunately, there is no content specific for your current browser. Please use the dropdown menu above to choose a different browser." | translate("no-content-for-platform-message", "paragraph") }}</p>

Powered by Google App Engine
This is Rietveld