kareila: (Default)
kareila ([personal profile] kareila) wrote in [site community profile] changelog2011-02-16 10:11 pm

[dw-free] editjournal.bml: missing UL on success page when entry's crossposted

[commit: http://hg.dwscoalition.org/dw-free/rev/f91104d03d57]

http://bugs.dwscoalition.org/show_bug.cgi?id=3445

Make sure <li> elements exist within <ul> tags, for display consistency.

Patch by [personal profile] ninetydegrees.

Files modified:
  • htdocs/editjournal.bml
--------------------------------------------------------------------------------
diff -r 0bd4eaaaff4a -r f91104d03d57 htdocs/editjournal.bml
--- a/htdocs/editjournal.bml	Wed Feb 16 20:32:10 2011 +0800
+++ b/htdocs/editjournal.bml	Wed Feb 16 16:10:51 2011 -0600
@@ -282,6 +282,7 @@ body<=
                                                             auth_response => $POST{"prop_xpost_resp_$acctid"}
                                                                              || $GET{"prop_xpost_resp_$acctid"}})
                                                       });
+                    $xpost_result .= "<ul>\n";
                     $xpost_result .= join("\n",
                                           map {"<li>"
                                                . BML::ml('xpost.request.success',
@@ -295,6 +296,8 @@ body<=
                                                           'editurl' => $edititemlink})
                                                . " </strong></div></li>"}
                                               @{$xpost_failures});
+                    $xpost_result .= "</ul>\n";
+                    $xpost_result .= "<br/>";
                 }
 
                 my $result = "<?h1 $ML{'.success.head'} h1?>";
--------------------------------------------------------------------------------