[Popcon-developers] popcon page renders like the old style of the main debian website

[Stéphane Blondon]

2014/1/25 Bill Allombert <Bill.Allombert at math.u-bordeaux.fr>:
>> If you disagree, it would be nice to convert the list with <pre> tags
>> to a real html list (<ul> ou <dl>). The patch is trivial, I can send
>> it if you want.
>
> Yes please. I have some computer issue right now.

The patch is attached to this e-mail.

I had an error when generated the html file but I think it's caused by
the way I test it and not the patch itself. (I used the archive you
provided me and overwrote the popcon.pl file with the lastest in Git).
The error:
printf() on closed filehandle DAT at ./popcon.pl line 263.

So I just commented the line to check the render of the html.

Is it ok for you?


Cheers,
-- 
Imprimez ce message en A2 et en couleur au moins 500 fois!
Brûlez des arbres!!

-- envoyé depuis ma centrale à charbon
Stéphane
-------------- next part --------------
--- popcon.orig.pl	2014-01-26 00:30:52.011359012 +0100
+++ popcon.pl	2014-01-26 00:30:31.720191275 +0100
@@ -144,6 +144,9 @@
     margin-bottom: 1em;
     padding-left: 1em;
   }
+  .legend dt{
+    margin: 0;
+  }
 </style>
 <div id="tabs">
   <ul>
@@ -165,14 +168,18 @@
 sub legend
 {
   print HTML <<EOF;
-<pre>
-inst     : number of people who installed this package;
-vote     : number of people who use this package regularly;
-old      : number of people who installed, but don't use this package regularly;
-recent   : number of people who upgraded this package recently;
-no-files : number of people whose entry didn't contain enough information (atime
-and ctime were 0).
-</pre>
+<dl class='legend'>
+    <dt>inst</dt>
+    <dd>number of people who installed this package</dd>
+    <dt>vote</dt>
+    <dd>number of people who use this package regularly</dd>
+    <dt>old</dt>
+    <dd>number of people who installed, but don't use this package regularly</dd>
+    <dt>recent</dt>
+    <dd>number of people who upgraded this package recently</dd>
+    <dt>no-files</dt>
+    <dd>number of people whose entry didn't contain enough information (atime and ctime were 0)</dd>
+</dl>
 EOF
 }