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

[Philip Romanik]

I'm seeing the problem on FF and IE (I tried FF 1.5 and IE6) on Windows. I 
updated subversion and tried again using swf but I still can't get the 
event changes to stick.

Phil


>Hrmm, I'll look into that.  Which environment(s) are you getting the
>"Apply" problem in?  It works for me in FF2.0 and Safari in OS X, and
>FF on XP, all in both SWF and DHTML.  I'll try it in IE6 and 7 on the
>PCs and get back to you.  Lemme check and make sure that all the
>modified files were included in the patch, as well...
>
>Sorry about the tabs/spaces... I forgot to convert.  Thanks for
>reminding me.
>
>
>-Josh
>
>
>On Jan 21, 2007, at 2:11 PM, Philip Romanik wrote:
>
> > 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
> >