[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

[Philip Romanik]

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 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>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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.openlaszlo.org/pipermail/laszlo-dev/attachments/20080708/7bff07fe/attachment.html