#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)
Change History (5)
by , 21 years ago
comment:1 by , 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 , 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 , 21 years ago
| Owner: | set to |
|---|---|
| Status: | new → closed |
comment:4 by , 5 years ago
| Component: | → Web |
|---|
Patch for styles.css file in web CVS module