doctool: Fix use of <dl> tag in templates

This isn't legal Mallard

4 files changed, 46 insertions, 29 deletions

diff --git a/giscanner/doctemplates/Python/enum.tmpl b/giscanner/doctemplates/Python/enum.tmpl
index 2624c425..35cdd439 100644
--- a/giscanner/doctemplates/Python/enum.tmpl
+++ b/giscanner/doctemplates/Python/enum.tmpl
@@ -1,11 +1,13 @@
 <%inherit file="/base.tmpl"/>
 <%block name="details">
 % if node.members:
-<dl>
+<terms>
 % for member in node.members:
-<dt><p>${node.name}.${member.name.upper()} :</p></dt>
-<dd>${formatter.format(node, member.doc)}</dd>
+<item>
+<title><code>${node.name}.${member.name.upper()}</code></title>
+${formatter.format(node, member.doc)}
+</item>
 % endfor
-</dl>
+</terms>
 % endif
 </%block>
diff --git a/giscanner/doctemplates/Python/function.tmpl b/giscanner/doctemplates/Python/function.tmpl
index 356619c3..d1b57f4e 100644
--- a/giscanner/doctemplates/Python/function.tmpl
+++ b/giscanner/doctemplates/Python/function.tmpl
@@ -39,15 +39,19 @@ ${', '.join((formatter.format_parameter_name(node, arg) for arg in formatter.get
 </%block>
 <%block name="details">
 % if formatter.get_parameters(node) or node.retval:
-<dl>
+<terms>
 % for ix, arg in enumerate(formatter.get_parameters(node)):
-<dt><p>${formatter.format_parameter_name(node, arg)} :</p></dt>
-<dd>${formatter.format(node, arg.doc)}</dd>
+<item>
+<title><code>${formatter.format_parameter_name(node, arg)}</code></title>
+${formatter.format(node, arg.doc)}
+</item>
 % endfor
 % if node.retval and node.retval.type.ctype != 'void':
-<dt><p>Returns :</p></dt>
-<dd>${formatter.format(node, node.retval.doc)}</dd>
+<item>
+<title><code>Returns</code></title>
+{formatter.format(node, node.retval.doc)}
+</item>
 % endif
-</dl>
+</terms>
 % endif
 </%block>
diff --git a/giscanner/doctemplates/Python/signal.tmpl b/giscanner/doctemplates/Python/signal.tmpl
index 0df13f64..fa850418 100644
--- a/giscanner/doctemplates/Python/signal.tmpl
+++ b/giscanner/doctemplates/Python/signal.tmpl
@@ -13,23 +13,30 @@ user_param1, ...)
 </code></synopsis>
 </%block>
 <%block name="details">
-<dl>
-<dt><p>${formatter.to_underscores(node.parent.name).lower()} :</p></dt>
-<dd><p>instance of ${namespace.name}.${node.parent.name} that is emitting the signal</p></dd>
+<terms>
+<item>
+<title><code>${formatter.to_underscores(node.parent.name).lower()}</code></title>
+<p>instance of ${namespace.name}.${node.parent.name} that is emitting the signal</p>
+</item>
 % for arg in formatter.get_parameters(node):
-<dt><p>${arg.argname} :</p></dt>
-<dd>${formatter.format(node, arg.doc)}</dd>
+<item>
+<title><code>${arg.argname}</code></title>
+${formatter.format(node, arg.doc)}
+</item>
 % endfor
-<dt><p>user_param1 :</p></dt>
-<dd><p>first user parameter (if any) specified with the connect() method</p></dd>
-<dt><p>... :</p></dt>
-<dd><p>additional user parameters (if any)</p></dd>
+<title><code>user_param1</code></title>
+<p>first user parameter (if any) specified with the connect() method</p>
+<item>
+<title><code>...</code></title>
+<p>additional user parameters (if any)</p>
+</item>
 % if node.retval and \
      node.retval.type.ctype != 'void' and \
      node.retval.type.ctype is not None:
-<dt><p>Returns :</p></dt>
-<dd>${node.retval.type.ctype} ${formatter.format(node, node.retval.doc)}</dd>
+<item>
+<title><code>Returns</code></title>
+<p>${node.retval.type.ctype} ${formatter.format(node, node.retval.doc)}</p>
+</item>
 % endif
-</dl>
+</terms>
 </%block>
-
diff --git a/giscanner/doctemplates/Python/vfunc.tmpl b/giscanner/doctemplates/Python/vfunc.tmpl
index c4716a5e..98a30932 100644
--- a/giscanner/doctemplates/Python/vfunc.tmpl
+++ b/giscanner/doctemplates/Python/vfunc.tmpl
@@ -15,15 +15,19 @@ ${', '.join((arg.argname for arg in formatter.get_parameters(node)))}\
 </%block>
 <%block name="details">
 % if formatter.get_parameters(node) or node.retval:
-<dl>
+<terms>
 % for arg in formatter.get_parameters(node):
-<dt><p>${arg.argname} :</p></dt>
-<dd>${formatter.format(node, arg.doc)}</dd>
+<item>
+<title><code>${arg.argname}</code></title>
+${formatter.format(node, arg.doc)}
+</item>
 % endfor
 % if node.retval and node.retval.type.ctype != 'void':
-<dt><p>Returns :</p></dt>
-<dd>${formatter.format(node, node.retval.doc)}</dd>
+<item>
+<title><code>Returns</code></title>
+${formatter.format(node, node.retval.doc)}
+</item>
 % endif
-</dl>
+</terms>
 % endif
 </%block>