[Laszlo-dev] For Review: 20080707-laszlo-8 Summary: IE7: DHTML: Calendar: Delete leaves an event without a title on the Calendar Regression: Clicking the close button in the event editor closes the event editor and doesn't remove the event

[J Crowley]

Change 20080707-laszlo-8 by laszlo at T43-L3XEXMW on 2008-07-07 17:36:24 EDT
    in /home/laszlo/src/svn/openlaszlo/trunk-liquid
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: IE7: DHTML: Calendar: Delete leaves an event
            without a title on the Calendar
         Regression: Clicking the close button in the
            event editor closes the event editor and
            doesn't remove the event

New Features:

Bugs Fixed: LPP-6532 - IE7: DHTML: Calendar: Delete
                leaves an event without a title on
                the Calendar
            LPP-5578 - Regression: Clicking the close
                button in the event editor closes the
                event editor and doesn't remove the
                event

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

Documentation:

Release Notes:

Details: For the IE bug, I changed the order of events
    and it seems to have fixed the issue.
    
    For the other bug, I added in a few checks to delete
    a new event if "apply" isn't clicked and the info
    panel is closed.

Tests: In IE7, select an event (it can sometimes be
    difficult to reproduce, but the "Pablo" event seems
    to exhibit the behavior the most often), click (i),
    and click "Delete" in the info panel.  There should
    no longer be an empty event lingering on the screen.
    
    For the other bug, create an event and then click
    the close button.  The event should disappear if
    "apply" isn't clicked.  If "apply" is clicked, the
    event should stay when you close the info panel.
    
    This shouldn't delete any events that have already
    been created, if you close the info panel without
    clicking "apply".

Files:
M      demos/calendar/cal-data.lzx
M      demos/calendar/calendar.lzx
M      demos/calendar/infopanel.lzx

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20080707-laszlo-8.tar