Yeah I agree, but unfortunately some browsers (Firefox) don't send an onload event for the initial load. setHTML() is only called after the iframe inits, and only if the src attribute is null: setHTML() won't apply if the src/iframe src is set. So it should be safe, I think...

On 10/8/10 3:43 PM, Henry Minsky wrote:

  approved...

One thing though, I'm a little scared of the hardcoded time delay, if
there were any way to interlock that
with a document body onload event or something, it would be more comforting.


On Fri, Oct 8, 2010 at 5:39 PM, Max Carlson <[email protected]
<mailto:[email protected]>> wrote:

    Change maxcarlson-20101008-hOI by [email protected] on
    2010-10-08 13:51:35 MDT
        in /Users/maxcarlson/openlaszlo/trunk2
        for http://svn.openlaszlo.org/openlaszlo/trunk

    Summary: Ensure the html.html attribute is properly set at startup time

    Bugs Fixed: LPP-9431 - html attribute of <html> element doesn't set
    the html

    Technical Reviewer: hminsky
    QA Reviewer: chelgeson

    Details: Delay initial setHTML() calls until after the page has a
    chance to load.  Improve documentation for html attribute.

    Tests: See LPP-9431

    Files:
    M       lps/components/extensions/html.lzx

    Changeset:
    http://svn.openlaszlo.org/openlaszlo/patches/maxcarlson-20101008-hOI.tar




--
Henry Minsky
Software Architect
[email protected] <mailto:[email protected]>


Reply via email to