Comment 45 for bug 1512093

Revision history for this message
kaputtnik (franku) wrote :

Regarding CSS: I think using the same text color for Functions and Links is confusing. Maybe just increase the font size of functions to 1.5em?
Not related to your changes: There is a double background for pre-tags and tags with class highlite. The 'highlite' background uses the whole width, the pre tag only the width of text (see screenshot). The pre tag is defined in base.css and we may want to overwrite it in documentation.css like:

pre {
 display: block; /*show black background over the whole width*/
}

Alternative is to display only the black background of pre without additional borders: Disable 'border' and 'background-color' for .highlite in documentation.css.

The militarysite documentation is really good now :-)

Animation:
lgtm :-) Since there are 'Mandatory' and 'Optional' statements, i think having them at most places would lead into less confusion. E.g. in <tribename>.lua something like "All table entries are mandatory."