Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(276)

Issue 4891044: Texi2HTML: don't wrap <p> around the contents of table cells. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 10 months ago by Reinhold
Modified:
7 years, 10 months ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Texi2HTML: don't wrap <p> around the contents of table cells. This fixes the ugly huge space around table cells in our documentation

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -0 lines) Patch
M Documentation/lilypond-texi2html.init View 2 chunks +9 lines, -0 lines 2 comments Download

Messages

Total messages: 3
Reinhold
This patch prevents all table cells in our documentation from being wrapped with a <p>, ...
7 years, 10 months ago (2011-08-13 13:51:42 UTC) #1
Neil Puttock
LGTM. http://codereview.appspot.com/4891044/diff/1/Documentation/lilypond-texi2html.init File Documentation/lilypond-texi2html.init (right): http://codereview.appspot.com/4891044/diff/1/Documentation/lilypond-texi2html.init#newcode2076 Documentation/lilypond-texi2html.init:2076: if (defined ($command_stack_at_begin->[0]) and $command_stack_at_begin->[0] eq 'multitable') whitespace ...
7 years, 10 months ago (2011-08-14 11:01:24 UTC) #2
Graham Percival (old account)
7 years, 10 months ago (2011-08-15 22:50:12 UTC) #3
LGTM; please push after fixing Neil's whitespace concerns.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b