git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] gitweb: blame: Minimize vertical table row padding
@ 2006-10-05 20:30 Luben Tuikov
  2006-10-06  7:45 ` Jakub Narebski
  0 siblings, 1 reply; 2+ messages in thread
From: Luben Tuikov @ 2006-10-05 20:30 UTC (permalink / raw)
  To: git

[-- Attachment #1: Type: text/plain, Size: 680 bytes --]

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(-)

[-- Attachment #2: 1207600725-p1.txt --]
[-- Type: text/plain, Size: 353 bytes --]

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


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] gitweb: blame: Minimize vertical table row padding
  2006-10-05 20:30 [PATCH] gitweb: blame: Minimize vertical table row padding Luben Tuikov
@ 2006-10-06  7:45 ` Jakub Narebski
  0 siblings, 0 replies; 2+ messages in thread
From: Jakub Narebski @ 2006-10-06  7:45 UTC (permalink / raw)
  To: git

Luben Tuikov wrote:

> 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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2006-10-06  7:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-10-05 20:30 [PATCH] gitweb: blame: Minimize vertical table row padding Luben Tuikov
2006-10-06  7:45 ` Jakub Narebski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).