retitle 584439 Re: roffit: [PATCH] Missing doctype (no closing tag for <p>) tags 584439 + patch pending forwarded 584439 <[email protected]> thanks
markus schnalke <[email protected]> writes: > Package: roffit > Version: 0.6+cvs20090507-1 > Severity: normal > Tags: upstream > > Roffit appears to insert > > <p class="level0"> > > at the beginning of each line in the HTML body but there are no > closing </p> tags. XHTML requires closing all tags: The output is missing a proper doctype, in this case HTML 4.01 where closing <p> is not part of the standard. The following patch makes the HTML pass http://validator.w3.org/ Daniel, would you apply this to CVS. Thanks, Jari
>From b28df33c969f11b83047d1942c51ba1f817bb000 Mon Sep 17 00:00:00 2001 From: Jari Aalto <[email protected]> Date: Fri, 4 Jun 2010 10:14:05 +0300 Subject: [PATCH] roffit: Add doctype Organization: Private Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Signed-off-by: Jari Aalto <[email protected]> --- roffit | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/roffit b/roffit index 3149f37..eccc010 100755 --- a/roffit +++ b/roffit @@ -443,6 +443,8 @@ my $title=sprintf("%s man page", if($standalone) { print $OutFH <<MOO +<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" + "http://www.w3.org/TR/html4/loose.dtd";> <html><head> <title>$title</title> <meta name="generator" content="roffit $version"> -- 1.7.1
