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

Delta Between Two Patch Sets: pages/acceptable-ads.raw

Issue 5440736082264064: Issue 2075 - Put meta tags inside of page header. (Closed)
Left Patch Set: Changes generated by fixed conversion script. Created March 8, 2015, 9:53 p.m.
Right Patch Set: Wrap meta tags in head even when head doesn't yet exist. Created March 9, 2015, 11:48 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Right: Side by side diff | Download
LEFTRIGHT
(no file at all)
1 title=title-full 1 title=title-full
2 2
3 3 <head><meta content="$s1$" name="title">
saroyanm 2015/03/09 12:51:48 nit: Please make both (head, meta) tags to be on t
4 <meta content="$s1$" name="title">
5 <meta content="$s2$" name="description"> 4 <meta content="$s2$" name="description">
6 <meta content="$s1$" property="og:title"> 5 <meta content="$s1$" property="og:title">
7 <meta content="$s2$" property="og:description"> 6 <meta content="$s2$" property="og:description">
7 </head>
8 8
9 <h2 id="info">$s5$</h2> 9 <h2 id="info">$s5$</h2>
10 10
11 <p> 11 <p>
12 $s6$ 12 $s6$
13 </p> 13 </p>
14 14
15 <p> 15 <p>
16 $s7(http://acceptableads.org)$ 16 $s7(http://acceptableads.org)$
17 </p> 17 </p>
(...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 <li> 149 <li>
150 $s47(/forum/viewforum.php?f=12)$ 150 $s47(/forum/viewforum.php?f=12)$
151 </li> 151 </li>
152 </ol> 152 </ol>
153 153
154 <h2 id="who">$s48$</h2> 154 <h2 id="who">$s48$</h2>
155 155
156 <p> 156 <p>
157 $s49(/blog/adblock-plus-user-survey-results-part-3)$ 157 $s49(/blog/adblock-plus-user-survey-results-part-3)$
158 </p> 158 </p>
LEFTRIGHT

Powered by Google App Engine
This is Rietveld