[Laszlo-dev] For Review: 20080620-laszlo-4 Summary: UPDATE: Fix scrollbar example, uncomment the code Jim had commented

[Max Carlson]

Approved!  I still see these warnings in 
/examples/components/grid_example.lzx?lzr=dhtml&debug=true :

They were there without your change, but can you file a bug?  Thanks!

DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
INFO @lz/scrollbar.lzx#67: .thumb.$lzc$set_visible(null) is deprecated. 
Perhaps you meant .thumb.$lzc$set_visible(false)? If not, use 
.thumb.setVisibility('collapse').
INFO: setNodeAttribute: p is null in binder @home_default
INFO: setNodeAttribute: p is null in binder @work_default
INFO: setNodeAttribute: p is null in binder @home_default
INFO: setNodeAttribute: p is null in binder @work_default
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
INFO: setNodeAttribute: p is null in binder @home_default
INFO: setNodeAttribute: p is null in binder @work_default
INFO: setNodeAttribute: p is null in binder @home_default
INFO: setNodeAttribute: p is null in binder @work_default
INFO: setNodeAttribute: p is null in binder @home_default
INFO: setNodeAttribute: p is null in binder @work_default
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
INFO: setNodeAttribute: p is null in binder @home_default
INFO: setNodeAttribute: p is null in binder @home_default
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136
DEBUG @base/basegridrow.lzx#4: Dynamic method? basegridrow['_updateIt'] 
= base/basegridrow.lzx#192/136



J Crowley wrote:
> Change 20080620-laszlo-4 by laszlo at T43-L3XEXMW on 2008-06-20 12:57:06 EDT
>    in /home/laszlo/src/svn/openlaszlo/trunk-six
>    for http://svn.openlaszlo.org/openlaszlo/trunk
> 
> Summary: UPDATE: Fix scrollbar example, uncomment the code Jim had 
> commented
> 
> New Features:
> 
> Bugs Fixed: LPP-6421 - Fix scrollbar example, uncomment the code Jim had 
> commented
> 
> Technical Reviewer: max
> QA Reviewer: ptw
> Doc Reviewer: (pending)
> 
> Documentation:
> 
> Release Notes:
> 
> Details: UPDATE: Per Tucker's recommendation, I changed
>    frosty.lzx back to a class, but enveloped it in a
>    <library/> this time.  Instead of modifying scrollbar_example.lzx,
>    then, I changed components_sampler.lzx to accommodate
>    the change to frosty.  Everything seems to work now,
>    in DHTML and SWF, and this has the added bonus of
>    eliminating the errors and warnings in scrollbar_example.lzx
>    that I'd originally modified frosty.lzx to fix.
>       I also modified window_example.lzx to reflect the change.  I
>    don't believe Frosty is used anywhere else.
>       I also de-commented the menubar Jim had commented out,
>    since that now actually works and should be brought back.
>       I also fixed an "undefined" warning that was showing up,
>    from gridcolumn.lzx.
> 
> Tests: UPDATE: Run /examples/components/scrollbar_example.lzx
>    in SWF and DHTML.  There should now be no warnings or
>    errors, and everything should run fine.  Also, run
>    /examples/components/component_sampler.lzx, which should
>    also run fine in both DHTML and SWF.
> 
> Files:
> M      lps/components/lz/gridcolumn.lzx
> M      examples/components/testmedia/frosty.lzx
> M      examples/components/window_example.lzx
> M      examples/components/component_sampler.lzx
> 
> Changeset: 
> http://svn.openlaszlo.org/openlaszlo/patches/20080620-laszlo-4.tar
> 
> 

-- 
Regards,
Max Carlson
OpenLaszlo.org