Commit 63fcbe00a66d7cb7d8bce9a36120a19a809a33b8 by Peter "Pasky" Baudis
"gitweb: Do not use absolute font sizes"
made gitweb use default font size, the font size you have configured
your browser to use, as a basis; other font sizes are given as
percentage.
David Kågedal noticed problem with this approach: too many web sites
explicitly select extra small font sizes, which means that you have to
select a "standard" font size in your browser that is a bit larger
than one would actually like.
Additionally, as Jan Hudec noticed, gitweb should explicitely request
being somewhat smaller than normal, because it has good use for long
lines. Normally one would usually want to set font so large, that
more than ~120 characters won't fit on the screen, since otherwise
pages that don't use margins are simply unreadable. However gitweb
presents a table with several columns, so having wider line is OK for
it.
Jan Hudec proposed to use 'font-size: small' to take above issues into
account. This avoid using "brain-damaged" explicit-pixel sizes, while
taking into account nature of gitweb output.
Thanks to mentioned commit by Pasky we need to add 'font-size: small'
in only one place, for BODY element.
See the original messages:
Message-ID: <87odkmgaj5.fsf@xxxxxxxxxxxxxx>
Message-ID: <20070515150912.GA3653@xxxxxxxxxxxxxxxx>
Proposed-by: Jan Hudec <bulb@xxxxxx>
Signed-off-by: Jakub Narebski <jnareb@xxxxxxxxx>
---
This is commit wich has commit message larger than patch itself, but
I think the explanation is as important as patch itself, perhaps even
more important.
gitweb/gitweb.css | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/gitweb/gitweb.css b/gitweb/gitweb.css
index b57c8be..02623cb 100644
--- a/gitweb/gitweb.css
+++ b/gitweb/gitweb.css
@@ -1,5 +1,6 @@
body {
font-family: sans-serif;
+ font-size: small;
border: solid #d9d8d1;
border-width: 1px;
margin: 10px;
--
1.5.1.4
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
