Opened 21 years ago

Closed 21 years ago

Last modified 6 years ago

#8288 closed patch

Fix PRE tag in styles.css

Reported by: sev- Owned by: fingolfin
Priority: normal Component: Web
Version: 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- 21 years ago.
Patch for styles.css file in web CVS module

Download all attachments as: .zip

Change History (5)

by sev-, 21 years ago

Attachment: patch added

Patch for styles.css file in web CVS module

comment:1 by fingolfin, 21 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, 21 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, 21 years ago

Owner: set to fingolfin
Status: newclosed

comment:4 by digitall, 6 years ago

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