threads / patch / 5821

patchgitweb: blame: Minimize vertical table row padding

Subject: [PATCH] gitweb: blame: Minimize vertical table row padding

## tl;dr

2 messages between Oct 5, 2006 and Oct 6, 2006. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Luben Tuikov· Oct 5, 2006, 20:30 UTC · lore

Minimize vertical table row padding for blame only. I discovered this while having the browser's blame output right next to my editor's window, only to notice how much vertically stretched the blame output was.

Blame most likely shows source code and is in this way more "spartan" than the rest of the tables gitweb shows.

This patch makes the blame table more vertically compact, thus being closer to what you'd see in your editor's window, as well as reusing more window estate to show more information (which in turn minimizes scrolling).

Signed-off-by: Luben Tuikov <ltuikov@yahoo.com>
---
 gitweb/gitweb.css |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)
Show changes to gitweb/gitweb.css +6 −0
diff --git a/gitweb/gitweb.css b/gitweb/gitweb.css
index eb9fc38..668e69a 100644
--- a/gitweb/gitweb.css
+++ b/gitweb/gitweb.css
@@ -173,6 +173,12 @@ table.blame {
 	border-collapse: collapse;
 }
 
+table.blame td {
+	padding: 0px 5px;
+	font-size: 12px;
+	vertical-align: top;
+}
+
 th {
 	padding: 2px 5px;
 	font-size: 12px;
-- 
1.4.2.3.gc45c-dirty
Jakub Narebski· Oct 6, 2006, 07:45 UTC · re: Luben Tuikov · lore

Re: [PATCH] gitweb: blame: Minimize vertical table row padding

Luben Tuikov wrote:
Show 12 quoted lines
> Minimize vertical table row padding for blame only.  I
> discovered this while having the browser's blame output
> right next to my editor's window, only to notice how much
> vertically stretched the blame output was.
> 
> Blame most likely shows source code and is in this way
> more "spartan" than the rest of the tables gitweb shows.
> 
> This patch makes the blame table more vertically compact,
> thus being closer to what you'd see in your editor's window,
> as well as reusing more window estate to show more
> information (which in turn minimizes scrolling).

Perhaps we should change blame output from table based to div block based, like ordinary "blob" view, hmmm...?

-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git

← back to recent threads