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

Unified Diff: pages/getting_started.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/getting_started.raw
diff --git a/pages/getting_started.raw b/pages/getting_started.raw
index 052b8b53fbe31ed0a55272016223ac971026a0d0..797e3a8c40602cd920121d0a9cd4a30e3186bf52 100644
--- a/pages/getting_started.raw
+++ b/pages/getting_started.raw
@@ -1,8 +1,8 @@
-
-<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="general">$s5$</h2>

Powered by Google App Engine
This is Rietveld