Comment 13 for bug 1173426

Revision history for this message
Doug Smythies (dsmythies) wrote :

For whatever reason, the yelp-xsl file html.xsl file uses text_light for a great many things that should be at least normal text weight. So the ubuntu.xsl files for both serverguide and ubuntu-docs have merely redefined text_light as actually the same as text. (well, it defines it as '333' instead of '333333', but it renders as the same thing. (how is a color defined as only 3 digits anyhow? It takes 6 to define the 3 bytes.)) There are many other redefinitions of colors in the ubuntu.xsl files, seemingly done to force the resulting html to mimic the light-moin-theme, but at the same time breaking some other things.

So the way I am proposing to solve this bug report is: Add the missing span.key stuff to the ubuntu.xsl files; Add back meaningful color definitions; Add override definitions to the ubuntu.xsl files to use those colors as required.

For example:

<xsl:param name="color.text_light" select="'#333333'"/>
<xsl:param name="color.text_light_actually" select="'#aaaaaa'"/>

/* overide the yelp html.xsl here. Why? Because the defination of text_light is wrong above, but fixing it breaks a multitude of other things. */
span.gui, span.guiseq { color: </xsl:text>
  <xsl:value-of select="$color.text_light_actually"/><xsl:text>; }

It is a bit of a kludge on top of a kludge, but I don't know what else to do. It also makes for a bit of a messy css file as things end up defined and then overridden later.