[#LPP-4001] roundrectbutton in dhtml

OpenLaszlo

roundrectbutton in dhtml

Created: 18/May/07 02:18 PM Updated: 27/Sep/07 08:56 AM

Component/s:

Components - Incubator

Affects Version/s:

Legals

Fix Version/s:

RingDing (4.1)

Time Tracking:

Original Estimate:

Not Specified

Remaining Estimate:

0 minutes

Time Spent:

1 hour

Severity:	Minor
Fixed in Change#:	5,145
Runtime:	N/A
Flags:	External
Fix in hand:	False

Description

« Hide

Initially reported by Geoff Crawford, geoff@innov8cs.com.

In dhtml, a canvas with only a roundrectbutton fails in 4.0.2, with the error
this.context has no properties. It works fine in swf.

See http://localhost:8080/legals/lps/components/incubator/test/test-roundrectbutton.lzx?lzr=dhtml&debug=true
for a simple example.

All

Comments

Work Log

Change History

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

Benjamin Shine - 18/May/07 03:50 PM

(Rewrote bug for clarity.)

[ Permlink | « Hide ]

Benjamin Shine - 18/May/07 04:10 PM

Isolating further, the problem comes down to calling drawview.clear() before the drawview is completely built. This code snippet demonstrates the problem; running in dhtml gives a debugger error,
ERROR: http://localhost:8080/legals/test/drawing/drawing.lzx?lzt=object&lzt=object&debug=true&lzr=dhtml&_canvas_debug=true:188: this.context has no properties

    <drawview width="200" height="200">
        <handler name="onwidth">
            <![CDATA[
            this.drawStructure();
            ]]>
        </handler>
        <method name="drawStructure">
            this.clear();
        </method>
    </drawview>

[ Permlink | « Hide ]

Benjamin Shine - 21/May/07 03:31 PM

This snippet will be fixed by ~~LPP-4007~~.

[ Permlink | « Hide ]

Benjamin Shine - 21/May/07 03:33 PM

This bug is just about the roundrect problem; the code snippet above is actually demonstrating a slightly separate problem, which is tracked in ~~LPP-4007~~.

[ Permlink | « Hide ]

Mamye Kratt - 07/Jun/07 10:22 AM

(local legals build r5334)
roundrect is broken, but removing changes to drawview made by Jim in changeset r5295 fixes the issue - it's a regression in drawview (spoke w/Max).

[ Permlink | « Hide ]

Jim Grandy - 07/Jun/07 10:27 AM

I don't see how a regression introduced after 4.0.2 shipped could possibly be the cause of the original bug.

[ Permlink | « Hide ]

Mamye Kratt - 07/Jun/07 12:25 PM

(local legals build r5334)
drawview issues need to be resolved to test this. changeset r5296 needs to be backed out.

[ Permlink | « Hide ]

Mamye Kratt - 28/Jun/07 10:42 AM

(4.0 branches (4.0.3)local build r5538)
Fixed

[ Permlink | « Hide ]

Mamye Kratt - 28/Jun/07 10:42 AM

Needs to be tested in legals branch.

[ Permlink | « Hide ]

Mamye Kratt - 17/Jul/07 12:03 PM

Need to test in legals.