[Laszlo-reviews] For Review: Change 20100605-bargull-iBh Summary: improve appearance of leading white-space in <programlisting>
andre.bargull at udo.edu
Tue Jun 8 08:38:18 PDT 2010
On 6/7/2010 4:42 PM, P T Withington wrote:
> I'm pretty sure the 1.08 is me, because I find that the tty font looks too small visually otherwise. I wasn't worried about the indenting, since it's a fixed-width font.
> If that is really of concern, I can only think of two other ways:
'of concern' in regard to find a different way to fix the indentation
bug or 'of concern' in regard to fix the bug at all?
> 1) adjust the tracking: letter-spacing: -0.08em
The spacing between characters becomes too narrow when the
letter-spacing css style is set to -0.08em for the complete <code>
element. If the style is just set for the <span> containing the leading
white-space, the visual result is the same as for setting the font-size
to 0.926em. I guess you've meant to propose the latter.
> 2) create a nested div for each level of indentation give those divs the same padding as the single line code
> On 2010-06-05, at 12:07, André Bargull wrote:
>> Do you know of a better way to handle this bug? IMO, this patch is a bit clumsy.
>> - André
>> Change 20100605-bargull-iBh by bargull at Bargull02 on 2010-06-05 17:37:38
>> in /home/anba/src/svn/openlaszlo/trunk
>> for http://svn.openlaszlo.org/openlaszlo/trunk
>> Summary: improve appearance of leading white-space in<programlisting>
>> Bugs Fixed: LPP-5990 (xslt for<programlisting> formats code blocks poorly)
>> Technical Reviewer: ptw
>> QA Reviewer: (pending)
>> The programlisting's code font-size is set to 1.08em to improve readability (see styles.css). But there arises one disadvantage:
>> increasing the font-size also increases the width of the leading white-space in each line. This is clearly not intended. So we'd like to tell the css to increase the font-size only after the leading white-space, but unfortunately this isn't possible (at least I don't know how to do this). This is how I worked around this problem:
>> - when the code is processed in lzx-pretty-print.xsl, I annotate all leading white-space with a<leading-space> element
>> - and in common-html.xsl, the<leading-space> element gets transformed to a simple<span> element with the "leading-space" css-class set
>> - the "leading-space" css-class has got a font-size of 0.926em, which is the reciprocal of 1.08, so it effectively reverts the increased font-size of code blocks
>> styles.css also adds padding for code blocks, but this is actually only useful for single-line code. Multi-line code are now marked with a "multiline" attribute, so it's possible to attach a different css-class to those elements (see common-html.xsl).
>> ant developers
>> open dguide chapters with programlisting
>> M docs/includes/styles.css
>> M docs/src/xsl/common-html.xsl
>> M docs/src/xsl/lzx-pretty-print.xsl
>> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20100605-bargull-iBh.tar
More information about the Laszlo-reviews