It's a bit quirky in swf but it's a good start

I think one change is needed in html.lzx. Change

    lz.Browser.callJS('lz.embed.iframemanager.setHTML', id, html);

to

lz.Browser.callJS('lz.embed.iframemanager.setHTML', null, id, html);

Another issue is if you hit the Load button when the string is empty. This redraws the page inside an iframe.

Otherwise approved!


On Thu, Jul 15, 2010 at 4:25 PM, Max Carlson wrote:

Change 20100715-maxcarlson-e by maxcarl...@friendly on 2010-07-15 13:17:47 PDT
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Add generic event/callback system, allow html component to have default, static text.

Bugs Fixed: LPP-9190 - Add generic events to html tag, LPP-9191 - Add `html` attribute to html component.

Technical Reviewer: promanik
QA Reviewer: hminsky

Details: extensions/html - ADd generic __iframecallback() method, use for loaded callbacks. Embedded iframes should be able to call window.parent.lz.embed.iframemanager[iframeid](name, value) to send an event. We need a way for the iframe to know its id, but this should give you something to play with. Added html setter, setHTML() proxy method for Flash.

iframemanager - Add setHTML() to set initial default HTML for the iframe, set only once, after the initial load. Update __gotload() to use lz.embed.iframemanager.__iframecallback().
examples/extensions/html - Add test for html attribute.

Tests: See LPP-9190, LPP-9191

Files:
M       lps/components/extensions/html.lzx
M       lps/includes/source/iframemanager.js
M       examples/extensions/html.lzx

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20100715-maxcarlson-e.tar

Reply via email to