git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 4/4] gitweb: make search help link less ugly

From
Tony Finch <dot@dotat.at>
Date
Jul 9, 2013, 17:15 UTC
Message-ID
<E1UyevC-0003du-K4@hermes-2.csi.cam.ac.uk>
In-Reply-To
<20130714234915.GB19057@google.com>

The search help link was a superscript question mark right next to a drop-down menu, which looks misaligned and is a cramped and awkward click target. Remove the superscript tags and add some spacing to fix these nits. Add a title attribute to provide an explanatory mouseover.

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/gitweb.perl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index c029b98..874c948 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -4029,9 +4029,9 @@ sub print_search_form {
 	      $cgi->input({-name=>"a", -value=>"search", -type=>"hidden"}) . "\n" .
 	      $cgi->input({-name=>"h", -value=>$search_hash, -type=>"hidden"}) . "\n" .
 	      $cgi->popup_menu(-name => 'st', -default => 'commit',
-	                       -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .
-	      $cgi->sup($cgi->a({-href => href(action=>"search_help")}, "?")) .
-	      " search:\n",
+			       -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .
+	      " " . $cgi->a({-href => href(action=>"search_help"),
+			     -title => "search help" }, "?") . " search:\n",
 	      $cgi->textfield(-name => "s", -value => $searchtext, -override => 1) . "\n" .
 	      "<span title=\"Extended regular expression\">" .
 	      $cgi->checkbox(-name => 'sr', -value => 1, -label => 're',
-- 
1.8.3.1.605.g85318f5
Previous: Tony FinchNext: Tony Finch
Message 7 of 12 in “gitweb: Ensure OPML text fits inside its box.”
  1. gitweb: Ensure OPML text fits inside its box.Tony Finch, Jul 9, 2013
  2. gitweb: vertically centre contents of page footerTony Finch, Jul 9, 2013
  3. Jonathan NiederJul 14, 2013
  4. 3/4 gitweb: omit the repository owner when it is unsetTony Finch, Jul 2, 2013
  5. 1/4 gitweb: Ensure OPML text fits inside its box.Tony Finch, Jul 9, 2013
  6. 2/4 gitweb: vertically centre contents of page footerTony Finch, Jul 9, 2013
  7. 4/4 gitweb: make search help link less uglyTony Finch, Jul 9, 2013
  8. Tony FinchJul 15, 2013
  9. Jakub NarębskiAug 3, 2013
  10. Tony FinchAug 5, 2013
  11. Jakub NarębskiAug 3, 2013
  12. Tony FinchAug 5, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.