Hello all,
On 23/09/06, George V. Reilly <[EMAIL PROTECTED]> wrote:
Edward L. Fox wrote:
>
> But we should change one thing before we include this patch into the
> official version. In the patch file, line 97:
>
> + execute "normal! A\npre { font-family: courier; color: " . s:fgc
> . "; background-color: " . s:bgc . "; }\e"
>
> Should be:
>
> + execute "normal! A\npre { font-family: Courier, monospace; color:
> " . s:fgc . "; background-color: " . s:bgc . "; }\e"
>
> As W3C suggested, every "font-family" indication must finish with a
> *GENERIC* font family name, possible values are "serif", "sans-serif"
> or "monospace". So I added "monospace" here.
>
Agreed. I didn't know this until now. I will update my patch later today.
I'd like to request that the wretchedly ugly Courier font *NOT* be
hardcoded in to 2html.vim. I always end up removing it when I use 2html.
The font-family should either be just "monospace" or allow a global
variable to set the font.
I favour the variable, but courier is a good default. Most books use
the courier font for verbatim text like code, it is the globally
recognized "this part is code" font.
Again I will update my patch and reply when done.
Best Regards
Edd
