Make the text-only results a list rather than a table. Makes more sense
that way (also tweaks styling slightly
Patch by kareila.
diffs (23 lines):
diff -r ac7055e44342 -r 7474d8f1cc7e cgi-bin/LJ/Directory/Results.pm
--- a/cgi-bin/LJ/Directory/Results.pm Tue Dec 28 20:56:53 2010 +0800
+++ b/cgi-bin/LJ/Directory/Results.pm Tue Dec 28 20:58:23 2010 +0800
@@ -91,15 +91,15 @@ sub render_simple {
my $updated = LJ::get_timeupdate_multi($self->userids);
- my $ret = "<table summary='' id='SearchResults' cellspacing='1'>";
+ my $ret = "<ul>";
foreach my $u (@users) {
- $ret .= "<tr><td class='SearchResult'>";
+ $ret .= "<li>";
$ret .= $u->ljuser_display . " - " . $u->name_html;
# FIXME: consider replacing this with $u->last_updated
$ret .= " <small>(Last updated: ". LJ::diff_ago_text(
$updated->{$u->id} ) . ")</small>";
- $ret .= "</td></tr>";
+ $ret .= "</li>";
}
- $ret .= "</table>";
+ $ret .= "</ul>";
return $ret;
}
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog
