html/show.tmpl: Always use <ul> for Maintainers

Even if we only have one. Just makes for a more consistent
display.

diff --git a/templates/html/show.tmpl b/templates/html/show.tmpl
index aa3435f18ac21c2071a04c895fd2dddb51e1f86d..544fe2ecb8895488bbd8ee222b194f3e8160b119 100644 (file)
--- a/templates/html/show.tmpl
+++ b/templates/html/show.tmpl
@@ -101,20 +101,16 @@
 
 [% IF maintainers.size == 1 -%]
   <h3>Maintainer:</h3>
 
 [% IF maintainers.size == 1 -%]
   <h3>Maintainer:</h3>

-  [%- FOREACH maintainers -%]
-       <a href="mailto:[% mail %]">[% name | html %]</a>
-       (<a href="[% ddpo_url _ mail %]" title="An overview over the maintainer's packages and uploads">QA Page</a>)
-  [%- END -%]

 [%- ELSE -%]
   <h3>Maintainers:</h3>
 [%- ELSE -%]
   <h3>Maintainers:</h3>

-  [%- FOREACH maintainers -%]
-    [%- '<ul>' IF loop.first -%]
+[%- END %]
+[%- FOREACH maintainers -%]
+  [%- '<ul>' IF loop.first -%]

        <li><a href="mailto:[% mail %]">[% name | html %]</a>
        (<a href="[% ddpo_url _ mail %]" title="An overview over the maintainer's packages and uploads">QA Page</a>)
        </li>
        <li><a href="mailto:[% mail %]">[% name | html %]</a>
        (<a href="[% ddpo_url _ mail %]" title="An overview over the maintainer's packages and uploads">QA Page</a>)
        </li>

-    [%- '</ul>' IF loop.last -%]
-  [%- END -%]
-[%- END %]
+  [%- '</ul>' IF loop.last -%]
+[%- END -%]

 
 [% url = page.get_newest('url');
    url = page.get_newest('homepage');
 
 [% url = page.get_newest('url');
    url = page.get_newest('homepage');