Jakub Narebski <jnareb@gmail.com> writes: > @@ -1559,6 +1562,7 @@ sub git_history_body { > next; > } > > + #my $ref = defined $refs ? format_ref_marker($refs, $commit) : ''; > my $ref = format_ref_marker($refs, $commit); > > if ($alternate) { Do you want to change this or not? Make up your mind. > + $paging_nav .= " ⋅ " . > + $cgi->a({-href => href(action=>"history", hash=>$hash, hash_base=>$hash_base, > + file_name=>$file_name, page=>$page-1), > + -accesskey => "p", -title => "Alt-p"}, "prev"); This is something you inherited from the original and not your fault, but I am not sure if these -title entries are of good taste (we have corresponding "Alt-n" for "next"). Something more descriptive like "<Alt-p> for 100 newer changes", perhaps? Also, "first", "prev" and "next" have always confused me. Maybe "latest", "older" and "newer" are better labels for them? Is 100 a good default? It feels a bit unbalanced compared to the height of the default summary page with descriptions, 16 shortlog entries, 16 tags and 12 heads. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Tue Aug 22 16:16:37 2006
This archive was generated by hypermail 2.1.8 : 2006-08-22 16:17:17 EST