Change S/L/T to summary/log/tree

In yet another attempt at better usability, the cryptic S/L/T links are
changed to show their full name.

Suggested-by: Kristian Høgsberg <krh@bitplanet.net>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>

3 files changed, 10 insertions, 9 deletions

diff --git a/cgit.css b/cgit.css
index 9c79c32..112dac1 100644
--- a/cgit.css
+++ b/cgit.css
@@ -350,58 +350,59 @@ table.diff td div.add {
 table.diff td div.del {
 	color: red;
 }
 
 .sha1 {
 	font-family: monospace;
 	font-size: 90%;
 }
 
 .left {
 	text-align: left;
 }
 
 .right {
 	text-align: right;
 }
 
 table.list td.repogroup {
 	font-style: italic;
 	color: #888;
 }
 
 a.button {
 	font-size: 80%;
-	color: #333;
-	background-color: #ccc;
-	border: solid 1px #999;
+	color: #aaa;
+	background-color: #eee;
+	border: solid 1px #aaa;
 	padding: 0em 0.5em;
 	margin: 0.1em 0.25em;
 }
 
 a.button:hover {
 	text-decoration: none;
-	background-color: #eee;
+	color: #333;
+	background-color: #ccc;
 }
 
 a.primary {
 	font-size: 100%;
 }
 
 a.secondary {
 	font-size: 90%;
 }
 
 td.toplevel-repo {
 
 }
 
 table.list td.sublevel-repo {
 	padding-left: 1.5em;
 }
 
 span.age-mins {
 	font-weight: bold;
 	color: #080;
 }
 
 span.age-hours {
diff --git a/ui-repolist.c b/ui-repolist.c
index d5c77ea..4f820a8 100644
--- a/ui-repolist.c
+++ b/ui-repolist.c
@@ -66,33 +66,33 @@ void cgit_print_repolist(struct cacheitem *item)
 	for (i=0; i<cgit_repolist.count; i++) {
 		cgit_repo = &cgit_repolist.repos[i];
 		if ((last_group == NULL && cgit_repo->group != NULL) ||
 		    (last_group != NULL && cgit_repo->group == NULL) ||
 		    (last_group != NULL && cgit_repo->group != NULL &&
 		     strcmp(cgit_repo->group, last_group))) {
 			html("<tr class='nohover'><td colspan='4' class='repogroup'>");
 			html_txt(cgit_repo->group);
 			html("</td></tr>");
 			last_group = cgit_repo->group;
 		}
 		htmlf("<tr><td class='%s'>",
 		      cgit_repo->group ? "sublevel-repo" : "toplevel-repo");
 		html_link_open(cgit_repourl(cgit_repo->url), NULL, NULL);
 		html_txt(cgit_repo->name);
 		html_link_close();
 		html("</td><td>");
 		html_ntxt(cgit_max_repodesc_len, cgit_repo->desc);
 		html("</td><td>");
 		html_txt(cgit_repo->owner);
 		html("</td><td>");
 		print_modtime(cgit_repo);
 		html("</td><td>");
 		html_link_open(cgit_repourl(cgit_repo->url),
-			       "Summary", "button");
-		html("S</a>");
-		cgit_log_link("L", "Log", "button", NULL, NULL, NULL);
-		cgit_tree_link("T", "Tree", "button", NULL, NULL, NULL);
+			       NULL, "button");
+		html("summary</a>");
+		cgit_log_link("log", NULL, "button", NULL, NULL, NULL);
+		cgit_tree_link("tree", NULL, "button", NULL, NULL, NULL);
 		html("</td></tr>\n");
 	}
 	html("</table>");
 	cgit_print_docend();
 }
diff --git a/ui-tree.c b/ui-tree.c
index 1037c82..b6cb813 100644
--- a/ui-tree.c
+++ b/ui-tree.c
@@ -71,49 +71,49 @@ static int ls_item(const unsigned char *sha1, const char *base, int baselen,
 		return 0;
 	}
 
 	html("<tr><td class='ls-mode'>");
 	html_filemode(mode);
 	html("</td><td>");
 	if (S_ISDIRLNK(mode)) {
 		htmlf("<a class='ls-mod' href='");
 		html_attr(fmt(cgit_repo->module_link,
 			      name,
 			      sha1_to_hex(sha1)));
 		html("'>");
 		html_txt(name);
 		html("</a>");
 	} else if (S_ISDIR(mode)) {
 		cgit_tree_link(name, NULL, "ls-dir", cgit_query_head,
 			       curr_rev, fullpath);
 	} else {
 		cgit_tree_link(name, NULL, "ls-blob", cgit_query_head,
 			       curr_rev, fullpath);
 	}
 	htmlf("</td><td class='ls-size'>%li</td>", size);
 
 	html("<td>");
-	cgit_log_link("L", "Log", "button", cgit_query_head, curr_rev,
+	cgit_log_link("log", NULL, "button", cgit_query_head, curr_rev,
 		      fullpath);
 	html("</td></tr>\n");
 	free(name);
 	return 0;
 }
 
 static void ls_head()
 {
 	html("<table class='list'>\n");
 	html("<tr class='nohover'>");
 	html("<th class='left'>Mode</th>");
 	html("<th class='left'>Name</th>");
 	html("<th class='right'>Size</th>");
 	html("<th/>");
 	html("</tr>\n");
 	header = 1;
 }
 
 static void ls_tail()
 {
 	if (!header)
 		return;
 	html("</table>\n");
 	header = 0;