Bug in Stylist incorrectly deals with hr styles (solution suggested)

[egk10@…]

A piece of 'lazy' code in stylist.js confuses h1, h2, h3, h4 etc. tags with hr (horizaontal rule). This causes a crash if the stylesheet contains a style such as hr.short (e.g. to produce a 75% horizontal rule). The affected code is the following:

~line 122 // If we don't have an ancestor, but it's a div/span/p/hx stle, we can make one

if(( tag == 'div' tag == 'span' tag == 'p'

(tag.substr(0,1) == 'h' && tag.length == 2) ))

the problem is that 'h' && tag.length == 2, which is intended to match h1 h2 h3 etc., but also matches hr. The solution is to replace the above code with:

if(( tag == 'div' tag == 'span' tag == 'p'

(tag.substr(0,1) == 'h' && tag.length == 2) ) && tag != 'hr' )

Hope this helps.

[gogo]

applied in changeset:181