More html compliance for the package page

debexpo/templates/package/index.mako

@@ -170,7 +170,7 @@
   % for comment in package_version.package_comments:
 
     <li>
-      <p>
+      <div>
         <pre>${ h.util.html_escape(comment.text) }</pre>
 
 % if comment.outcome == c.constants.PACKAGE_COMMENT_OUTCOME_NEEDS_WORK:
@@ -191,7 +191,7 @@
 
 % endif
 
-     </p>
+     </div>
    </li>
 
   % endfor
@@ -206,9 +206,9 @@
 
 % if 'user_id' in c.session:
 <h4>New comment</h4>
-<fieldset>
 ${ h.html.tags.form(h.url('comment', packagename=c.package.name), method='post') }
-${ h.html.tags.hidden('package_version', package_version.id) }
+<fieldset>
+${ h.html.tags.hidden('package_version', package_version.id, id="package_version_%d" % package_version.id) }
 
 % if hasattr(c, 'form_errors'):
     <% c.form_errors %>
@@ -217,25 +217,25 @@ ${ h.html.tags.hidden('package_version', package_version.id) }
 <table>
     <tr>
         <td>Comment</td>
-        <td>${ h.html.tags.textarea('text', cols=82, rows=10) }</td>
+        <td>${ h.html.tags.textarea('text', cols=82, rows=10, id="text_%d" % package_version.id) }</td>
     </tr>
     <tr>
         <td>Outcome</td>
-        <td>${ h.html.tags.select('outcome', c.constants.PACKAGE_COMMENT_OUTCOME_UNREVIEWED, c.outcomes) }</td>
+        <td>${ h.html.tags.select('outcome', c.constants.PACKAGE_COMMENT_OUTCOME_UNREVIEWED, c.outcomes, id="outcomes_%d" % package_version.id) }</td>
     </tr>
 % if config['debexpo.debian_specific'] == 'true' and c.user.status == c.constants.USER_STATUS_DEVELOPER:
     <tr>
         <td>${ _('Uploaded to Debian') }</td>
-        <td>${ h.html.tags.checkbox('status') }</td>
+        <td>${ h.html.tags.checkbox('status', id="status_%d" % package_version.id) }</td>
     </tr>
 % endif
     <tr>
-        <td>${ h.html.tags.submit('commit', _('Submit')) }</td>
+        <td>${ h.html.tags.submit('commit', _('Submit'), id="commit_%d" % package_version.id) }</td>
     </tr>
 </table>
+</fieldset>
 ${ h.html.tags.end_form() }
 
-</fieldset>
 
 % endif