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

Side by Side Diff: includes/meta/standard.tmpl

Issue 29622568: Issue 6013 - Add site name to document title in website-defaults standard metadata (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Use config.has_option, use title_exclude_sitename Created Nov. 28, 2017, 4:28 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | pages/index.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <meta charset="utf-8"> 1 <meta charset="utf-8">
2 <meta http-equiv="x-ua-compatible" content="ie=edge"> 2 <meta http-equiv="x-ua-compatible" content="ie=edge">
3 <meta name="viewport" content="width=device-width, initial-scale=1"> 3 <meta name="viewport" content="width=device-width, initial-scale=1">
4
5 {% if title_exclude_sitename or config.has_option("general", "sitename") == Fals e %}
Vasily Kuznetsov 2017/11/30 12:53:28 The second part of this condition can be written a
ire 2017/11/30 13:20:00 I like your suggestion better. Done.
4 <title>{{ title | translate("page-title", "Page title") }}</title> 6 <title>{{ title | translate("page-title", "Page title") }}</title>
7 {% else %}
8 <title>{{ title | translate("page-title", "Page title") }} | {{ config.get("gene ral", "sitename") }}</title>
9 {% endif %}
10
5 {% if description %} 11 {% if description %}
6 <meta name="description" content="{{ description | translate("page-description ", "Page description") }}"> 12 <meta name="description" content="{{ description | translate("page-description ", "Page description") }}">
7 {% endif %} 13 {% endif %}
8 <link rel="canonical" href="{{ get_canonical_url(page) }}"> 14 <link rel="canonical" href="{{ get_canonical_url(page) }}">
OLDNEW
« no previous file with comments | « no previous file | pages/index.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld