Hi Derrel,

Thanks for your answer Derrell.

We are extening the default table resize behaviour in our software. In 0.7 this was no problem, as all the member variables used to be protected. In qooxdoo 0.8 all the member variables are now private (which is all fine to protected the internals of the class, to make it immutable).  In our case we needed this special protected getter method, so that we can still extend the resize behaviour, without writing our own colomnmodel / resize behaviour.

Tino


-----Original Message-----
> Date: Wed, 29 Jul 2009 17:01:54 +0200
> Subject: Re: [qooxdoo-commit] SF.net SVN: qooxdoo:[19717]
> trunk/qooxdoo/framework/source/class/qx/ui/  table/columnmodel
> From: Derrell Lipman <derrell.lip...@unwireduniverse.com>
> To: t...@users.sourceforge.net

>
Use of protected methods is generally deprecated,so if this change is really necessary, please change it to be a public method. I'm very curious, though, why you want/need this information. It's used for calculating widths. The post-calculation data is already available elsewhere, and if the user messes with it pre-calculation, it's likely to cause random errors. Without further understanding of the need of this, I'd opt not to include this new method. It's just a bit too potentially dangerous without clear advantage. Please enlighten me. :-)

Derrell


On Wed, Jul 29, 2009 at 10:52, <t...@users.sourceforge.net> wrote:
Revision: 19717
         http://qooxdoo.svn.sourceforge.net/qooxdoo/?rev=19717&view=rev
Author:   tbtz
Date:     2009-07-29 14:52:26 +0000 (Wed, 29 Jul 2009)

Log Message:
-----------
Added protected method to table resize behavior + fixed a bug when using a custom resize behavior

...


Modified: trunk/qooxdoo/framework/source/class/qx/ui/table/columnmodel/resizebehavior/Default.js
===================================================================
--- trunk/qooxdoo/framework/source/class/qx/ui/table/columnmodel/resizebehavior/Default.js      2009-07-29 14:25:55 UTC (rev 19716)
+++ trunk/qooxdoo/framework/source/class/qx/ui/table/columnmodel/resizebehavior/Default.js      2009-07-29 14:52:26 UTC (rev 19717)
@@ -549,6 +549,17 @@
          (width - (widthUsed - tableColumnModel.getColumnWidth(lastCol)));
        tableColumnModel.setColumnWidth(lastCol, newWidth);
      }
+    },
+
+
+    /**
+     * Returns the data of all the columns that are visible and will be resized.
+     *
+     * @return {Map[]} the resize column data.
+     */
+    _getResizeColumnData : function()
+    {
+      return this.__resizeColumnData;
    }
  },



This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
trial. Simplify your report design, integration and deployment - and focus on
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
qooxdoo-commit mailing list
qooxdoo-com...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-commit



--
The trick to education is to teach people in such a way that they don't realize they're learning until it's too late.
 - Harold Eugene "Doc" Edgerton ("Poppa Flash"), 1903-1990, fabled MIT professor and inventor of the stroboscopic flash

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to