ubuntu-bots team mailing list archive

[Thomas Molloy <beetyrootey@xxxxxxxxx>]

Thomas Molloy has proposed merging lp:~lderan/ubuntu-bots/meeetingology-output into lp:~ubuntu-bots/ubuntu-bots/meetingology.

Requested reviews:
  Ubuntu IRC Bots (ubuntu-bots)

For more details, see:
https://code.launchpad.net/~lderan/ubuntu-bots/meeetingology-output/+merge/174539

Fixed the output for moinmoin output when there are unassigned actions, also removed the line between vote results, adding in line linking for the html output as well
-- 
https://code.launchpad.net/~lderan/ubuntu-bots/meeetingology-output/+merge/174539
Your team Ubuntu IRC Bots is requested to review the proposed merge of lp:~lderan/ubuntu-bots/meeetingology-output into lp:~ubuntu-bots/ubuntu-bots/meetingology.

=== modified file 'css-log-default.css'
--- css-log-default.css	2010-09-02 19:16:45 +0000
+++ css-log-default.css	2013-07-13 10:55:30 +0000
@@ -13,3 +13,6 @@
 body .topicline { color: #000080; font-weight: bold }
 body .cmd       { color: #007020; font-weight: bold }
 body .cmdline  { font-weight: bold }
+
+a {text-decoration:none;}
+a:hover {text-decoration:underline;}

=== modified file 'writers.py'
--- writers.py	2013-04-21 19:25:19 +0000
+++ writers.py	2013-07-13 10:55:30 +0000
@@ -456,8 +456,8 @@
                                (html(m2.group(1)),html(m2.group(2))))
                 if m2 is None:
                     outline = html(line)
-                lines.append('<a name="l-%(lineno)s"></a>'
-                             '<span class="tm">%(time)s</span>'
+                lines.append('<a href="#l-%(lineno)s" name="l-%(lineno)s">'
+                             '<span class="tm">%(time)s</span></a>'
                              '<span class="nk">%(nick)s</span> '
                              '%(line)s'%{'lineno':lineNumber,
                                          'time':html(m.group('time')),
@@ -1216,7 +1216,7 @@
         Votes.append(self.heading('Vote results'))
         for m in M.votes:
             #differentiate denied votes somehow, strikethrough perhaps?
-            Votes.append("\n * "+m)
+            Votes.append(" * "+m)
             motion = "Deadlock"
             if(M.votes[m][0] > M.votes[m][1]):
                 motion = "Motion carried"
@@ -1264,12 +1264,12 @@
                 m.assigned = True
         # unassigned items:
         Unassigned = [ ]
-        Unassigned.append("* **UNASSIGNED**")
+        Unassigned.append(" * **UNASSIGNED**")
         numberUnassigned = 0
         for m in M.minutes:
             if m.itemtype != "ACTION": continue
             if getattr(m, 'assigned', False): continue
-            Unassigned.append(" ** %s"%moin(m.line))
+            Unassigned.append("  * %s"%moin(m.line))
             numberUnassigned += 1
         if numberUnassigned == 0:
             Unassigned.append(" * (none)")