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

Unified 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: Created Nov. 28, 2017, 9:11 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 | no next file » | 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,8 +1,14 @@
<meta charset="utf-8">
<meta http-equiv="x-ua-compatible" content="ie=edge">
<meta name="viewport" content="width=device-width, initial-scale=1">
+
+{% if page == "index" or config.get("general", "sitename") is not defined %}
ire 2017/11/28 09:12:49 This currently doesn't work because if I try "conf
juliandoucette 2017/11/28 14:49:21 I don't think that the index page will be the only
juliandoucette 2017/11/28 14:49:21 I think that this is how to find the answer to you
ire 2017/11/28 16:11:26 Alright, thanks!
ire 2017/11/28 16:11:26 Acknowledged.
<title>{{ title | translate("page-title", "Page title") }}</title>
+{% else %}
+<title>{{ title | translate("page-title", "Page title") }} | {{ config.get("general", "sitename") }}</title>
+{% endif %}
+
{% 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 | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld