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

Unified Diff: pages/acceptable-ads.raw

Issue 5440736082264064: Issue 2075 - Put meta tags inside of page header. (Closed)
Patch Set: Wrap meta tags in head even when head doesn't yet exist. Created March 9, 2015, 11:48 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: pages/acceptable-ads.raw
diff --git a/pages/acceptable-ads.raw b/pages/acceptable-ads.raw
index ee9749cf1a41f9bc7c23de4564bd0888b501fbf3..be6434e1657ef880a57cb7ec5b5d2cfb20ceeb33 100644
--- a/pages/acceptable-ads.raw
+++ b/pages/acceptable-ads.raw
@@ -1,10 +1,10 @@
title=title-full
-
-<meta content="$s1$" name="title">
+<head><meta content="$s1$" name="title">
saroyanm 2015/03/09 12:51:48 nit: Please make both (head, meta) tags to be on t
<meta content="$s2$" name="description">
<meta content="$s1$" property="og:title">
<meta content="$s2$" property="og:description">
+</head>
<h2 id="info">$s5$</h2>
« no previous file with comments | « includes/index.tmpl ('k') | pages/acceptable-ads-manifesto.raw » ('j') | pages/bugs.raw » ('J')

Powered by Google App Engine
This is Rietveld