Use a span, rather than a div for inline blocks. Use
LJ::page_change_getargs rather than BML::self_link as the default handler.
Patch by kareila.
diffs (42 lines):
diff -r 7158b7960bfe -r 42861785f55c cgi-bin/DW/Controller/Search/Interests.pm
--- a/cgi-bin/DW/Controller/Search/Interests.pm Thu Jan 13 13:45:17 2011 +0800
+++ b/cgi-bin/DW/Controller/Search/Interests.pm Thu Jan 13 13:49:11 2011 +0800
@@ -318,14 +318,12 @@ sub interest_handler {
&& $should_show->( $_ ) # and should show to
the remote user
} values %$us;
my $navbar;
- my $self_link = sub { LJ::page_change_getargs( page => $_[0] ) };
my $results =
LJ::user_search_display( users => \@ul,
timesort => 1,
perpage => 50,
curpage => exists $args->{page} ?
$args->{page} : 1,
- self_link => $self_link,
navbar => \$navbar );
$rv->{int_users} = { count => scalar( @ul ), navbar => $navbar,
diff -r 7158b7960bfe -r 42861785f55c cgi-bin/weblib.pl
--- a/cgi-bin/weblib.pl Thu Jan 13 13:45:17 2011 +0800
+++ b/cgi-bin/weblib.pl Thu Jan 13 13:49:11 2011 +0800
@@ -399,7 +399,7 @@ sub paging_bar {
my ( $page, $pages, $opts ) = @_;
my $self_link = $opts->{self_link} ||
- sub { BML::self_link( { page => $_[0] } ) };
+ sub { LJ::page_change_getargs( page => $_[0] ) };
my $href_opts = $opts->{href_opts} || sub { '' };
@@ -428,9 +428,9 @@ sub paging_bar {
}
$nav .= "$left ";
- $nav .= "<div style='display: inline; text-align: center'>";
+ $nav .= "<span style='text-align: center'>";
$nav .= join ' ', @pagelinks;
- $nav .= "</div>";
+ $nav .= "</span>";
$nav .= " $right";
return "<div class='action-box'><div class='inner'>$nav</div></div><div
class='clear-floats'></div>\n";
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog
