History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: LPP-5385
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: P2 P2
Assignee: Unassigned
Reporter: André Bargull
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
OpenLaszlo

alert-control misses include

Created: 28/Jan/08 09:54 AM   Updated: 30/Jan/08 02:30 PM
Component/s: Components - LZ
Affects Version/s: RingDing (4.1)
Fix Version/s: RingDing (4.1)

Time Tracking:
Not Specified

Severity: Minor
Fixed in Change#: 7,916
Fixed in branch: trunk
Runtime: N/A
Fix in hand: True


 Description  « Hide
alert.lzx:

was:
<include href="lz/windowpanel.lzx" />

but must be:
<include href="lz/modaldialog.lzx" />


testapp:
<canvas debug="true" >
  <alert id="foo" />
</canvas>

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
André Bargull - 28/Jan/08 03:09 PM
send to laszlo-dev, will commit tomorrow, no "real" review is needed in this case...

André Bargull - 29/Jan/08 12:58 PM
------------------------------------------------------------------------
r7916 | bargull | 2008-01-29 21:56:54 +0100 (Tue, 29 Jan 2008) | 26 lines
Ge?\195?\164nderte Pfade:
   M /openlaszlo/trunk/lps/components/lz/alert.lzx

Change 20080128-bargull-8 by bargull@dell--p4--2-53 on 2008-01-28 23:55:40
    in /home/Admin/src/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: replace include-tag

New Features:

Bugs Fixed: LPP-5385 "alert-control misses include"

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

Documentation:

Release Notes:

Details:
replaced include of "windowpanel" with include of "modaldialog".


Tests:
see bugreport


------------------------------------------------------------------------

Mamye Kratt - 30/Jan/08 02:28 PM
(trunk 4 local build r7930)
test case ran successfully in dhtml and swf with debug on and off.

<canvas debug="true" >
  <alert id="foo" />
</canvas>

Mamye Kratt - 30/Jan/08 02:30 PM
Also visually checked WEB-INF/lps/misc/lzx-autoincludes.properties.