Opened 16 years ago

Closed 16 years ago

Last modified 11 months ago

#8288 closed patch

Fix PRE tag in styles.css

Reported by: sev- Owned by: fingolfin
Priority: normal Component: Web
Keywords: Cc:
Game:

Description

Now PRE HTML tag is rendered in Arial.

Although it is not a W3C violation as it _recommends_
use of monospaced font for this tag, as it is used on
code conventions page, it makes difficult to see exact
indentation rules.

This patch defines font for PRE as monospaced.

If this page is not applicable (PRE tag is also used in
signature on status page), then another solution would
be to change PRE tags to CODE in
web/docs/conventions.xml file.

Ticket imported from: #822332. Ticket imported from: patches/393.

Attachments (1)

patch (1.1 KB ) - added by sev- 16 years ago.
Patch for styles.css file in web CVS module

Download all attachments as: .zip

Change History (5)

by sev-, 16 years ago

Attachment: patch added

Patch for styles.css file in web CVS module

comment:1 by fingolfin, 16 years ago

The patch isn't acceptable in this form because it won't work on
system w/o "Lucida Console" (e.g. mine :-).

comment:2 by fingolfin, 16 years ago

Hrm, I am embarassed, I somehow overlooked the monospace
specified. Well that's better of course. I'll check this in.

Of course right now our page is rather, hrm, baroque. Like we are
using <font> elements etc. - I really would like to rewrite this to
use proper HTML 4.01 strict. But that's another matter :-)

comment:3 by fingolfin, 16 years ago

Owner: set to fingolfin
Status: newclosed

comment:4 by digitall, 11 months ago

Component: Web
Note: See TracTickets for help on using tickets.