RichText.js can throw error if lastChild is a text node

[andyhot@…]

See ​http://svn.dojotoolkit.org/dojo/trunk/src/widget/RichText.js line 376.

if (tmpContent.firstChild && tmpContent.firstChild.style) {
    tmpContent.firstChild.style.marginTop = 
        this._firstChildContributingMargin+"px";
    tmpContent.lastChild.style.marginBottom = 
        this._lastChildContributingMargin+"px";			
}

There's a check for firstStyle supporting the style attribute, but there's no similar check for the lastChild.

Without this fix, RichText? cannot edit html content (i.e. stored in a DB) which ends in a text node, so '<b>Hello</b> World' breaks, while

'<p><b>Hello</b> World</p>' works.

[andyhot@…]

Fix for this issue

[alex]

looks to have been already fixed, but [3912] should seal the deal.

[(none)]

Milestone 0.3release deleted