[PATCH] doc/style: use default font color specified by user

Yuya Nishihara yuya at tcha.org
Wed Nov 11 06:11:02 CST 2009


Re-sent with CC-ing, sorry.

Martin Geisler wrote:
> > # HG changeset patch
> > # User Yuya Nishihara <yuya at tcha.org>
> > # Date 1257857290 -32400
> > # Node ID d28887355d1436580708d3799d1b3329676de0df
> > # Parent  0d65e191ac4c2a3246fd12543819f3ccdd6a8a73
> > doc/style: use default font color specified by user
> >
> > * it's bad to specify only foreground color:
> >   http://www.w3.org/QA/Tips/color
> > * some people prefer dark background
> > * `color: #111' is mostly the same as `color: black',
> >   which is the default of almost all popular browsers.
> >
> > so it's preferable to delete `color: #111', rather than adding
> > `background-color: white'.
> 
> We would only have to add it to the body element, right?

Yeah, it can be, but I personally don't like such solution to overwrite
user's preference. If we design a large attractive website,
it's hard to go without forcing color/bgcolor, etc., but
this is really small, just a man page.

> > -    color: #111;  /* `inherit' is not supported by IE6 */
> > +    color: inherit;  /* NOTE: `inherit' is not supported by IE6 */
> >  }
> 
> You've updated the comment, is that because IE6 is no longer important?

Yes, we need to care about IE6. I mean the color of section headings are
unfortunately light blue because of IE6's rotten implementation.
If the color of seciton headings are really important, we need to set
`color: #111' with `background-color: white'.

And it's also preferable to add the following lines in case you try
printing them with your laser printer:

@media print {
    body, a:link.toc-backref, a:visited.toc-backref { color: black; }
}


Yuya


More information about the Mercurial-devel mailing list