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

Fri, 20 Jun 2008 18:17:37 -0700

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 [EMAIL PROTECTED] 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

Reply via email to