<html>
<body>
Approved, <br><br>
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.<br><br>
<br>
<blockquote type=cite class=cite cite="">Change 20080707-laszlo-8 by
laszlo@T43-L3XEXMW on 2008-07-07 17:36:24 EDT<br>
in /home/laszlo/src/svn/openlaszlo/trunk-liquid<br>
for
<a href="http://svn.openlaszlo.org/openlaszlo/trunk">
http://svn.openlaszlo.org/openlaszlo/trunk</a><br>
<br>
Summary: IE7: DHTML: Calendar: Delete leaves an event<br>
without a title on the Calendar<br>
Regression: Clicking the
close button in the<br>
event
editor closes the event editor and<br>
doesn't remove the event<br>
<br>
New Features:<br>
<br>
Bugs Fixed: LPP-6532 - IE7: DHTML: Calendar: Delete<br>
leaves an event without a title on<br>
the Calendar<br>
LPP-5578 - Regression: Clicking the close<br>
button in the event editor closes the<br>
event editor and doesn't remove the<br>
event<br>
<br>
Technical Reviewer: promanik<br>
QA Reviewer: max<br>
Doc Reviewer: (pending)<br>
<br>
Documentation:<br>
<br>
Release Notes:<br>
<br>
Details: For the IE bug, I changed the order of events<br>
and it seems to have fixed the issue.<br>
<br>
For the other bug, I added in a few checks to
delete<br>
a new event if "apply" isn't clicked and the
info<br>
panel is closed.<br>
<br>
Tests: In IE7, select an event (it can sometimes be<br>
difficult to reproduce, but the "Pablo"
event seems<br>
to exhibit the behavior the most often), click
(i),<br>
and click "Delete" in the info panel.
There should<br>
no longer be an empty event lingering on the
screen.<br>
<br>
For the other bug, create an event and then click<br>
the close button. The event should disappear
if<br>
"apply" isn't clicked. If
"apply" is clicked, the<br>
event should stay when you close the info panel.<br>
<br>
This shouldn't delete any events that have
already<br>
been created, if you close the info panel without<br>
clicking "apply".<br>
<br>
Files:<br>
M demos/calendar/cal-data.lzx<br>
M demos/calendar/calendar.lzx<br>
M demos/calendar/infopanel.lzx<br>
<br>
Changeset: <br>
<a href="http://svn.openlaszlo.org/openlaszlo/patches/20080707-laszlo-8.tar">
http://svn.openlaszlo.org/openlaszlo/patches/20080707-laszlo-8.tar</a><br>
<br>
</blockquote></body>
</html>