Approved, but there are issues in IE7/dhtml though. If you run one of your examples, you get an error:

'runtime' is null or not an object

This is coming from iframemanager.js:

            var metadata = lz.embed[iframe.owner]
            if (metadata.runtime == 'swf') {



Change 20080523-maxcarlson-8 by [EMAIL PROTECTED] on 2008-05-23
17:53:17 PDT
     in /Users/maxcarlson/openlaszlo/trunk-clean
for <http://svn.openlaszlo.org/openlaszlo/trunk>http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Make html tag act more like a view

New Features: scrollbars attribute can be set to false to hide scrollbars. targets are still watched for placement and width/height changes. In DHTML, iframes are placed inside the view, allowing them to be clipped, placed and have their z-order manipulated with
bringToFront() and sendToBack().

Bugs Fixed: LPP-4302 - Laszlo '<html>' Tag creates IFrame whose zIndex overlaps other controls

Technical Reviewer: promanik
QA Reviewer: <mailto:[EMAIL PROTECTED]>[EMAIL PROTECTED]
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details: LzSprite.js - Set div id for lzcanvasclickdiv for easy lookup. Call LzMouseKernel.__resetMouse() if a laszlo div is moused over.

LzMouseKernel.js - Add disableMouseTemporarily() API to disable global mouse events until a laszlo div is moused over. Added
setGlobalClickable() API to turn global mouse events on/off.

test/html.lzx - Use new view syntax with constraints to size the html tag correctly. Turn history off..

html.lzx - Proxy scrollbars and z values in create() and setPosition(). Add proxy scrollBy(). html now extends view. Add scrollbars attribute which if false will prevent scrollbars on the iframe at init time only. In DHTML, attach the iframe to the sprite and set the default z order to 0. In updatePos() for dhtml, read the
view x and y directly and otherwise read it relative to the canvas.
For dhtml, set the z based on the html view, otherwise base it on the
z of the target.  Turn on clickable to catch mouse events in dhtml.
bringToFront() and sendToBack() set the z based on the view z for dhtml, otherwise call bringToFront() in the iframemanager. Add scrollBy() method to set the scroll position of the iframe. Add mouse events for dhtml to enable/disable global mouse events and activate iframes.

Tests: lps/components/extensions/test/html.lzx?lzr=dhtml&lzt=html and lps/components/extensions/test/html-swf.jsp run as before. See the bug report for a test case

Files:
M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
M      WEB-INF/lps/lfc/kernel/dhtml/LzMouseKernel.js
M      lps/components/extensions/test/html.lzx
M      lps/components/extensions/html.lzx
M      lps/includes/source/iframemanager.js

Changeset: <http://svn.openlaszlo.org/openlaszlo/patches/20080523-maxcarlson-8.tar>http://svn.openlaszlo.org/openlaszlo/patches/20080523-maxcarlson-8.tar

Reply via email to