From 6fa849338755dbadd95700f72f85e7014dac8d3a Mon Sep 17 00:00:00 2001
From: Don Armstrong <don@donarmstrong.com>
Date: Thu, 1 Feb 2018 16:57:07 -0800
Subject: [PATCH] change buginfo to use a list instead of paragraphs

---
 templates/en_US/cgi/bugreport_buginfo.tx | 139 ++++++++++++++---------
 1 file changed, 87 insertions(+), 52 deletions(-)

diff --git a/templates/en_US/cgi/bugreport_buginfo.tx b/templates/en_US/cgi/bugreport_buginfo.tx
index 6349540..9067959 100644
--- a/templates/en_US/cgi/bugreport_buginfo.tx
+++ b/templates/en_US/cgi/bugreport_buginfo.tx
@@ -1,54 +1,89 @@
 <div class="buginfo">
-    <p>Reported by: <: raw(package_links(submitter=>$status.originator )) :></p>
-    <p>Date: <: $status.date_text :></p>
-    : if defined($status.owner) && $status.owner.length() {
-    <p>Owned by: <: raw(package_links("owner",$status.owner)) :></p>
-    : }
-    <p>Severity: <: if $status.severity { :><em class="severity"><: } :><: $status.severity :>
-        <: if $status.severity { :></em><: } :></p>
-    <p><: if $status.tags_array { :>Tags: <: $status.tags_array.join(' ') :></p> <: } :>
-    : if $status.mergedwith_array.count > 0 {
-    <p>Merged with <: bug_links(bug=>$status.mergedwith_array).join(",\n") :></p>
-    : }
-    : if $status.found_versions.count {
-    <p>Found in
-        version<:- if $status.found_versions.count > 1 { -:>s<: } -:>
-        <: $status.found_versions.join(', ') :></p>
-    : }
-    : if $status.fixed_versions.count {
-    <p>Fixed in 
-        version<:- if $status.fixed_versions.count > 1 { -:>s<: } -:>
-        <: $status.fixed_versions.join(', ') :></p>
-    : }
-    <: if $status.done.length() { :>
-    <p><strong>Done:</strong> <: $status.done :></p>
-    <: } :>
-    <: if $status.blockedby_array.count { :>
-    <p>Fix blocked by
-        : for $status.blockedby_array -> $bug {
-        <: bug_links("bug",$bug.bug_num) :>: <: $bug.subject :><: if !$~bug.is_last { :>,
-        <:  } else {  :>.<: } :>
-        : }
-    </p>
-    <: } :>
-    : if $status.blocks_array.count {
-    <p>Blocking fix for
-        : for $status.blocks_array -> $bug {
-        <: bug_links("bug",$bug.bug_num) :>: <: $bug.subject :><:if ! $~bug.is_last { :>,
-        <:  } else {  :>.<: } :>
-        : }
-    </p>
-    : }
-    : if $status.archived {
-    <p>Bug is archived. No further changes may be made.<p>
-    : }
-    : if defined $status.forwarded and $status.forwarded.length() {
-    <p>Forwarded to <: split($status.forwarded,',\s+').map(maybelink).join(', ') :></p>
-    : }
-    : if defined $status.summary and $status.summary.length() {
-    <p>Summary: <: $status.summary :></p>
-    : }
-    : if defined $status.outlook and $status.outlook.length() {
-    <p>Summary: <: $status.outlook :></p>
-    : }
+    <ul>
+        <li><span class="key">Reported by</span>
+            <span class="value"><: package_links(submitter=>$status.originator) :></span>
+        </li>
+        <li><span class="key">Date</span>
+            <span class="value"><: $status.date_text :></span>
+        </li>
+        : if defined($status.owner) && $status.owner.length() {
+        <li><span class="key">Owned by</span>
+            <span class="value"><: package_links("owner",$status.owner) :></span>
+        </li>
+        : }
+        <li><span class="key">Severity</span>
+            <span class="value">
+                <:- if $status.severity { :><em class="severity"><: } -:>
+                <:- $status.severity -:>
+                <:- if $status.severity { :></em><: } -:>
+            </span>
+        </li>
+        : if $status.tags_array {
+        <li><span class="key">Tags</span>
+            <span class="value"><: $status.tags_array.join(' ') :></span>
+        </li>
+        : }
+        : if $status.mergedwith_array.count > 0 {
+        <li><span class="key">Merged with</span>
+            <span class="value"><: bug_links(bug=>$status.mergedwith_array).join(",\n") :></span>
+        </li>
+        : }
+        : if $status.found_versions.count {
+        <li><span class="key">Found in
+            version<:- if $status.found_versions.count > 1 { -:>s<: } -:></span>
+            <span class="value"><: $status.found_versions.join(', ') :></span>
+        </li>
+        : }
+        : if $status.fixed_versions.count {
+        <li><span class="key">Fixed in 
+            version<: if $status.fixed_versions.count > 1 { :>s<: } :></span>
+            <span class="value"><: $status.fixed_versions.join(', ') :></span>
+        </li>
+        : }
+        <: if $status.done.length() { :>
+        <li><span class="key">Done</span>
+            <span class="value"><: $status.done :></span>
+        </li>
+        : }
+        : if $status.blockedby_array.count {
+        <li><span class="key">Fix blocked by</span>
+            <span class="value">
+                : for $status.blockedby_array -> $bug {
+                <: bug_links("bug",$bug.bug_num) :>: <: $bug.subject -:>
+                <:- if !$~bug.is_last { :>, <:  } else {  :>.<: } :>
+                : }
+            </span>
+        </li>
+        : }
+        : if $status.blocks_array.count {
+        <li><span class="key">Blocking fix for</span>
+            <span class="value" 
+                  : for $status.blocks_array -> $bug {
+                  <: bug_links("bug",$bug.bug_num) :>: <: $bug.subject -:>
+                  <:- if ! $~bug.is_last { :>, <:  } else {  :>.<: } :>
+                  : }
+            </span>
+        </li>
+        : }
+        : if $status.archived {
+        <li><span class="key">Bug is</span>
+            <span class="value">Archived</span>
+        </li>
+        : }
+        : if defined $status.forwarded and $status.forwarded.length() {
+        <li><span class="key">Forwarded to</span>
+            <span class="value"><: split($status.forwarded,',\s+').map(maybelink).join(', ') :></span>
+        </li>
+        : }
+        : if defined $status.summary and $status.summary.length() {
+        <li><span class="key">Summary</span>
+            <span class="value"><: $status.summary :></span>
+        </li>
+        : }
+        : if defined $status.outlook and $status.outlook.length() {
+        <li><span class="key">Outlook</span>
+            <span class="value"><: $status.outlook :></span>
+        </li>
+        : }
+    </ul>
 </div>
-- 
2.39.2