You need to fix line 222 of LzDrawView.lzs. It doesn't compile as written. And then approved.

Is this supposed to work on IE? It works great on Opera and Firefox. I couldn't test on Safari. On IE it has runtime errors:

In IE6: Object doesn't support this property or method
on this line:
this.__drawcontext=this.__LZcanvas.getContext("2d")

On IE7: 'undefined' is null or not an object. I don't have the line number.


Change 20070309-maxcarlson-B by [EMAIL PROTECTED] on 2007-03-09
19:15:13 PST
     in /Users/maxcarlson/openlaszlo/legals-clean

Fix drawview in dhtml

New Features: Drawview is now consistent across Safary 2.0.4, webkit,
firefox and opera 9.

Bugs Fixed:

Technical Reviewer: promanik
QA Reviewer: jcrowley
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details: drawing.lzx - relatively consistent behavior between swf and
dhtml.

LzSprite.js - Rely on css_hide_canvas_during_init to set display of
lzcanvasdiv, lzcanvasclickdiv.  Add
safari_visibility_instead_of_display quirk to ensure canvas tags show
up.  Add more safety checking for LzKeyboardKernel, LzInputTextSprite
LzMouseKernel and other properties.  Ie leak checking uses
onbeforeunload instead of onunload event.

LzMouseKernel.js - Safety checking for checking for LzInputTextSprite.

LzDrawView.lzs - Add complete implementation for DHTML (except IE).

excanvas.js - Include excanvas for ie.

embednew.js - Write a script tag out in dhtmlEmbedLFC for IE to load
excanvas.js.

Tests:

Files:
M      test/drawing/drawing.lzx
M      WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
M      WEB-INF/lps/lfc/kernel/dhtml/LzMouseKernel.js
M      WEB-INF/lps/lfc/views/platform/dhtml/LzDrawView.lzs
A      lps/includes/excanvas.js
M      lps/includes/source/embednew.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070309-
maxcarlson-B.tar

Reply via email to