includes/meta/standard.tmpl - Issue 29622568: Issue 6013 - Add site name to document title in website-defaults standard metadata

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

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: Created Nov. 28, 2017, 9:11 a.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
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 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 This currently doesn't work because if I try "config.get("general", "sitename")" where there is no sitename, I get a 500 error. I can't seem to figure out how to check if `config.get("general", "sitename")` exists without breaking things. Can you help? juliandoucette 2017/11/28 14:49:21 I don't think that the index page will be the only I don't think that the index page will be the only page that we want to exclude the sitename from. I suggest that we use an optional page attribute instead e.g. {% if title_exclude_sitename or ... juliandoucette 2017/11/28 14:49:21 I think that this is how to find the answer to you Show quoted text On 2017/11/28 09:12:49, ire wrote: > This currently doesn't work because if I try "config.get("general", "sitename")" > where there is no sitename, I get a 500 error. I can't seem to figure out how to > check if `config.get("general", "sitename")` exists without breaking things. Can > you help? I think that this is how to find the answer to your question (connect the dots): - Use grep (or similar) to find "config" in the "cms" repository - https://hg.adblockplus.org/cms/file/tip/cms/sources.py#l44 - https://hg.adblockplus.org/cms/file/tip/cms/sources.py#l70 - https://hg.adblockplus.org/cms/file/tip/cms/sources.py#l18 - https://docs.python.org/2/library/configparser.html - https://docs.python.org/2/library/configparser.html#ConfigParser.SafeConfigPa... - https://docs.python.org/2/library/configparser.html#configparser-objects - https://docs.python.org/2/library/configparser.html#ConfigParser.RawConfigParser - https://docs.python.org/2/library/configparser.html#ConfigParser.RawConfigPar... ire 2017/11/28 16:11:26 Alright, thanks! Show quoted text On 2017/11/28 14:49:21, juliandoucette wrote: > On 2017/11/28 09:12:49, ire wrote: > > This currently doesn't work because if I try "config.get("general", > "sitename")" > > where there is no sitename, I get a 500 error. I can't seem to figure out how > to > > check if `config.get("general", "sitename")` exists without breaking things. > Can > > you help? > > I think that this is how to find the answer to your question (connect the dots): > > - Use grep (or similar) to find "config" in the "cms" repository > - https://hg.adblockplus.org/cms/file/tip/cms/sources.py#l44 > - https://hg.adblockplus.org/cms/file/tip/cms/sources.py#l70 > - https://hg.adblockplus.org/cms/file/tip/cms/sources.py#l18 > - https://docs.python.org/2/library/configparser.html > - > https://docs.python.org/2/library/configparser.html#ConfigParser.SafeConfigPa... > - https://docs.python.org/2/library/configparser.html#configparser-objects > - > https://docs.python.org/2/library/configparser.html#ConfigParser.RawConfigParser > - > https://docs.python.org/2/library/configparser.html#ConfigParser.RawConfigPar... Alright, thanks! ire 2017/11/28 16:11:26 Acknowledged. Show quoted text On 2017/11/28 14:49:21, juliandoucette wrote: > I don't think that the index page will be the only page that we want to exclude > the sitename from. I suggest that we use an optional page attribute instead e.g. > {% if title_exclude_sitename or ... Acknowledged.
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) }}">

OLD	NEW

« no previous file with comments | « no previous file | no next file » | no next file with comments »