[Laszlo-dev] For Review: Change 20070121-jcrowley-E Summary: Cannot edit text fields in calendar demo; Problem occurs

[Philip Romanik]

Hi Josh,

I can't finish the review until a PC-build problem is fixed (I can't see 
all the graphics displayed in dhtml). However, I do have a few notes:

- Change tabs to spaces. Subversion will complain about it if you don't.
- I could not save changes by hitting the Apply button in swf. After 
hitting apply I clicked on another event and then I clicked on the event I 
just edited. My changes are lost. I tried both the title and the notes field.

Phil


>Change 20070121-jcrowley-E by jcrowley at doctormanhattan.mshome.net on
>2007-01-21 05:07:15 EST
>      in /Users/jcrowley/src/svn/openlaszlo/legals-four/demos/calendar
>
>Summary: Cannot edit text fields in calendar demo; Problem occurs
>         when moving an event in the weekly view of the calendar
>         demo; DHTML, IE 6: Graphical irregularity near scrollbar
>         when viewing maximized day in monthly view; various other
>         fixes and testing other bugs
>
>New Features: Added an "Apply" button to info panel to apply any
>         information changes or additions.
>
>Bugs Fixed: LPP-2986, LPP-2987, LPP-3014, LPP-3013, LPP-3222
>         (and thoroughly tested bugs LPP-2856, LPP-2854, LPP-2876,
>         LPP-3015, and LPP-3014, but was either unable to
>         reproduce them, or traced them back to non-Calendar-
>         specific issues.  Some were possibly fixed in the process
>         of fixing other bugs.)
>
>Technical Reviewer: promanik
>QA Reviewer: (pending)
>Doc Reviewer: (pending)
>
>Documentation:
>
>Release Notes: Addition of an "Apply" button in the info panel.
>         This allows users to apply any data changes in the info
>         panel, now that the fields can actually be edited.  Also
>         gives a clearer idea of when data is being changed and
>         applied, especially the even title, which would change
>         on an onblur event.
>
>Details: Resolved issues depicted in "summary".  Added an "apply"
>         button to allow users to change data, and have a clearer
>         idea of when data changes are applied.  Fixed text box
>         problems, and linked info text boxes to datasets.  Fixed
>         issues with moving days to other weeks and months in
>         weekly and monthly views, but am still working on getting
>         rid of some of the glitches that happen in day view.
>         Have filed this as a new bug, LPP-3450.  Have also filed
>         a couple other new bugs, some non-Calendar-specific, as
>         the result of working on this.  Have also thoroughly
>         tested several of the Calendar bugs and have found either
>         no reproducibility, or have traced them to non-Calendar-
>         specific issues.
>
>
>Tests: Ran Calendar app on various platforms, and discovered that
>         the fixes and changes have had the intended effects.
>
>Files:
>M      cal-data.lzx
>M      calendar.lzx
>M      infopanel.lzx
>M      event.lzx
>M      eventselector.lzx
>Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070121-
>jcrowley-E.tar