From patchwork Sat Feb  9 13:00:39 2013
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [5,of,6] minirst: HTML formatter tweaks
From: Dan Villiom Podlaski Christiansen <danchr@gmail.com>
X-Patchwork-Id: 863
Message-Id: <45b34ea40358b6f8fe14.1360414839@s0-0.paconsult7.bbnplanet.net>
To: Mercurial Devel <mercurial-devel@selenic.com>
Date: Sat, 09 Feb 2013 13:00:39 +0000

# HG changeset patch
# User Dan Villiom Podlaski Christiansen  <danchr@gmail.com>
# Date 1360414106 0
# Node ID 45b34ea40358b6f8fe14ef874ebbc5de5556f23c
# Parent  675a399c2c9d5c94d4798bc8fd11f855eaff5672
minirst: HTML formatter tweaks

- output table rows on distinct lines
- don't make the first row a table header

diff --git a/mercurial/minirst.py b/mercurial/minirst.py
--- a/mercurial/minirst.py
+++ b/mercurial/minirst.py
@@ -546,11 +546,8 @@ def formathtml(blocks):
             for row in table:
                 l = []
                 for v in zip(row):
-                    if not t:
-                        l.append('<th>%s</th>' % v)
-                    else:
-                        l.append('<td>%s</td>' % v)
-                t.append(' <tr>%s</tr>\n' % ''.join(l))
+                    l.append('<td>%s</td>' % v)
+                t.append(' <tr>%s</tr>\n' % '\n'.join(l))
             out.append('<table>\n%s</table>\n' % ''.join(t))
         elif btype == 'definition':
             openlist('dl', level)
diff --git a/tests/test-minirst.py.out b/tests/test-minirst.py.out
--- a/tests/test-minirst.py.out
+++ b/tests/test-minirst.py.out
@@ -758,9 +758,15 @@ 30 column format:
 html format:
 ----------------------------------------------------------------------
 <table>
- <tr><th>a</th><th>b</th><th>c</th></tr>
- <tr><td>1</td><td>2</td><td>3</td></tr>
- <tr><td>foo</td><td>bar</td><td>baz this list is very very very long man</td></tr>
+ <tr><td>a</td>
+<td>b</td>
+<td>c</td></tr>
+ <tr><td>1</td>
+<td>2</td>
+<td>3</td></tr>
+ <tr><td>foo</td>
+<td>bar</td>
+<td>baz this list is very very very long man</td></tr>
 </table>
 ----------------------------------------------------------------------