Approved,

The UI is a bit screwy no matter how you look at it. At first I was going to suggest adding an 'Are you Sure' dialog box when the user closes the infopanel. However, this doesn't solve all the UI issues.


Change 20080707-laszlo-8 by [EMAIL PROTECTED] on 2008-07-07 17:36:24 EDT
    in /home/laszlo/src/svn/openlaszlo/trunk-liquid
for <http://svn.openlaszlo.org/openlaszlo/trunk>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>http://svn.openlaszlo.org/openlaszlo/patches/20080707-laszlo-8.tar


Reply via email to