comparison tests/test-hgweb-empty.t @ 24041:9756f943634f

hgweb: use css margin instead of empty <p> before <div class="atom-logo"> The <p> elements were used to create an empty space between the last menu item (i.e. "help") and the atom feed icon, but they don't have any semantic meaning, so it is better to use css instead. The css rule uses top margin of 10px, which is equal to the top margin of the menu blocks ("help", "changeset, browse", etc). Previously, with <p> elements, the margin wasn't set explicitly and was browser-dependent. This change is a "better version" of 70cfa7e1611b, where <p> elements were simply properly closed.
author Anton Shestakov <engored@ya.ru>
date Thu, 05 Feb 2015 19:24:35 +0800
parents c4f6fc4eb01e
children fdf7794be41d
comparison
equal deleted inserted replaced
24040:7f375d2de945 24041:9756f943634f
46 46
47 </ul> 47 </ul>
48 <ul> 48 <ul>
49 <li><a href="/help">help</a></li> 49 <li><a href="/help">help</a></li>
50 </ul> 50 </ul>
51 <p></p>
52 <div class="atom-logo"> 51 <div class="atom-logo">
53 <a href="/atom-log" title="subscribe to atom feed"> 52 <a href="/atom-log" title="subscribe to atom feed">
54 <img class="atom-logo" src="/static/feed-icon-14x14.png" alt="atom feed" /> 53 <img class="atom-logo" src="/static/feed-icon-14x14.png" alt="atom feed" />
55 </a> 54 </a>
56 </div> 55 </div>
156 155
157 </ul> 156 </ul>
158 <ul> 157 <ul>
159 <li><a href="/help">help</a></li> 158 <li><a href="/help">help</a></li>
160 </ul> 159 </ul>
161 <p></p>
162 <div class="atom-logo"> 160 <div class="atom-logo">
163 <a href="/atom-log" title="subscribe to atom feed"> 161 <a href="/atom-log" title="subscribe to atom feed">
164 <img class="atom-logo" src="/static/feed-icon-14x14.png" alt="atom feed" /> 162 <img class="atom-logo" src="/static/feed-icon-14x14.png" alt="atom feed" />
165 </a> 163 </a>
166 </div> 164 </div>
262 <li><a href="/file/000000000000">browse</a></li> 260 <li><a href="/file/000000000000">browse</a></li>
263 </ul> 261 </ul>
264 <ul> 262 <ul>
265 <li><a href="/help">help</a></li> 263 <li><a href="/help">help</a></li>
266 </ul> 264 </ul>
267 <p></p>
268 <div class="atom-logo"> 265 <div class="atom-logo">
269 <a href="/atom-log" title="subscribe to atom feed"> 266 <a href="/atom-log" title="subscribe to atom feed">
270 <img class="atom-logo" src="/static/feed-icon-14x14.png" alt="atom feed" /> 267 <img class="atom-logo" src="/static/feed-icon-14x14.png" alt="atom feed" />
271 </a> 268 </a>
272 </div> 269 </div>