[PATCH v2 0/3] viewvcs: commit => query improvements

[Eric Wong <e@80x24.org>]

Threw in some minor formatting and HTML fixups, as well.
Most notably, there is consistently a <hr> before the textarea.

Eric Wong (3):
  viewvcs: close the <pre> tag if no query is extracted
  viewvcs: use wider textarea for search query
  viewvcs: generate search query for merge commits

 lib/PublicInbox/ViewVCS.pm | 75 ++++++++++++++++++++++++++++----------
 1 file changed, 55 insertions(+), 20 deletions(-)

Range-diff against v1:
-:  -------- > 1:  62e2c882 viewvcs: close the <pre> tag if no query is extracted
-:  -------- > 2:  f4b73a39 viewvcs: use wider textarea for search query
1:  8e9b1865 ! 3:  b49fa73f viewvcs: generate search query for merge commits
    @@ lib/PublicInbox/ViewVCS.pm: sub ibx_url_for {
     +			$in_titles = 1;
     +		} elsif ($in_titles) { # commit titles
     +			if (s/^  //) { # break up Xapian phrases
    -+				$t = join ' AND ', map { qq{s:"$_"} }
    ++				$t = join " AND\n ", map { qq{s:"$_"} }
     +					split /["\x{201c}\x{201d}]+/;
     +				push @s, $t if $t ne 's:"..."';
     +			} else { # trailing text or trailers?
    @@ lib/PublicInbox/ViewVCS.pm: committer $co
      	if (-s $fh > $MAX_SIZE) {
     @@ lib/PublicInbox/ViewVCS.pm: EOM
      		# commit?
    + 		print $zfh '</pre>';
      		my ($rows, $q) = PublicInbox::View::dfqry_text $ctx, $s;
    - 		if ($rows) {
    +-		if ($rows) {
     -			my $ibx_url = ibx_url_for($ctx);
     -			my $alt;
     -			if (defined $ibx_url) {
    @@ lib/PublicInbox/ViewVCS.pm: EOM
     -				$ibx_url = $upfx;
     -				$alt = '';
     -			}
    - 			print $zfh <<EOM;
    - </pre><hr><form action="$ibx_url"
    +-			print $zfh <<EOM;
    ++		print $zfh <<EOM if $rows;
    + <hr><form action="$ibx_url"
      id=related><pre>find related emails, including ancestors/descendants/conflicts
    -@@ lib/PublicInbox/ViewVCS.pm: id=related><pre>find related emails, including ancestors/descendants/conflicts
    + <textarea name=q cols=78 rows=$rows>$q</textarea>
    + <input type=submit value="search$alt"
    + />\t(<a href="${ibx_url}_/text/help/">help</a>)</pre></form>
      EOM
    - 		}
    - 	}
    +-		}
    ++	}
     +	if ($merge_titles) {
     +		print $zfh <<EOM;
    -+<form action="$ibx_url" id=merged><pre>find merged patch emails
    ++<hr><form action="$ibx_url" id=merged><pre>find merged patch emails
     +<textarea name=q cols=78
     +EOM
     +		my $nr = scalar @$merge_titles;
    @@ lib/PublicInbox/ViewVCS.pm: id=related><pre>find related emails, including ances
     +/></pre></form>
     +EOM
     +		undef $merge_titles;
    -+	}
    + 	}
      	chop($x = <<EOM);
      <hr><pre>glossary
    - --------