Re: [RFC/PATCH 2/8] docbook: improve css style

From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Mar 24, 2009, 00:20 UTC
Message-ID: <94a0d4530903231720r620e44fu90dd77a3231bd4d1@mail.gmail.com>
In-Reply-To: <20090323064242.GB1119@coredump.intra.peff.net>

On Mon, Mar 23, 2009 at 8:42 AM, Jeff King <peff@peff.net> wrote:

Show 10 quoted lines

> On Sun, Mar 22, 2009 at 08:05:15PM +0200, Felipe Contreras wrote:
>
>>  tt.literal, code.literal {
>>    color: navy;
>> +  font-size: 1em;
>> +}
>
> Isn't 1em already the default size? Or are you trying to override some
> other size specification elsewhere? It's hard to tell what the goal is
> because your commit message merely says "improve".

I've updated the CSS. Can you take a look again?

I changed the font-size to normal, except for the code chunks. Also, I changed the font of the in-paragrah code tags to sans-serif, that's the most sane way I can think to fix the problem with different font-size configured for monospace font.

I also reverted the line-height change.

--- a/Documentation/docbook-xsl.css
+++ b/Documentation/docbook-xsl.css
@@ -15,9 +15,8 @@ body blockquote {

 html body {
   margin: 1em 5% 1em 5%;
-  line-height: 1em;
+  line-height: 1.2;
   font-family: sans-serif;
-  font-size: small;
 }

 body div {
@@ -130,7 +129,7 @@ body pre {

 tt.literal, code.literal {
   color: navy;
-  font-size: 1em;
+  font-family: sans-serif;
 }

 code.literal:before { content: "'"; }
@@ -148,6 +147,7 @@ div.literallayout p {

 div.literallayout {
   font-family: monospace;
+  font-size: small;
   margin: 0em;
   color: navy;
   border: 1px solid silver;

-- 
Felipe Contreras