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

Unified Diff: includes/meta/standard.tmpl

Issue 29630648: Issue 6013 - Add site name to document title in website-defaults standard metadata (Closed) Base URL: https://hg.adblockplus.org/website-defaults
Patch Set: Remove repeated markup Created Dec. 13, 2017, 9:38 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
« no previous file with comments | « no previous file | pages/index.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: includes/meta/standard.tmpl
===================================================================
--- a/includes/meta/standard.tmpl
+++ b/includes/meta/standard.tmpl
@@ -1,14 +1,18 @@
<meta charset="utf-8">
<meta http-equiv="x-ua-compatible" content="ie=edge">
<meta name="viewport" content="width=device-width, initial-scale=1">
-{% if title_exclude_sitename or not has_string("name", "site") %}
-<title>{{ title | translate("page-title", "Page title") }}</title>
-{% else %}
-<title>{{ title | translate("page-title", "Page title") }} | {{ get_string("name", "site") }}</title>
-{% endif %}
+<title>
+ {{ title | translate("title", "Page title") }}
+
juliandoucette 2017/12/13 15:01:10 NIT: Unnecessary empty line?
ire 2017/12/13 19:16:00 Done.
+ {% if title_suffix and title_suffix != 'none' %}
+ | {{ title_suffix | translate("title-suffix", "Page title suffix") }}
+ {% elif has_string("name", "site") and title_suffix != 'none' %}
+ | {{ get_string("name", "site") }}
+ {% endif %}
+</title>
{% if description %}
<meta name="description" content="{{ description | translate("page-description", "Page description") }}">
{% endif %}
<link rel="canonical" href="{{ get_canonical_url(page) }}">
« 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