Thanks, Phil!  No problem, re: delay.

The black box thing is another bug I'm working on. There are some weird IE-specific visual glitches that are going on.

Thanks again for the review.  :)

-Josh

On Jan 28, 2007, at 1:25 PM, Philip Romanik wrote:

Hi Josh,

I'm able to see the buttons and other images again and things work much better. Approved! Sorry about the long delay.

I do see some strange behavior in IE6/IE7 when dragging a scrollbar when I expand a day in the month view. In IE7 I can't page down with the scroll bars. In IE6, I can, but there is a dark box that scrolls below the day. I'll wait a day (my system is a bit out of sync) and try again.

Phil


Change 20070121-jcrowley-E by [EMAIL PROTECTED] 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


Reply via email to