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