Yeah - one could just call into the other. But I'm thinking that this only belongs in TableView, not TableViewHeader.

On Aug 26, 2009, at 5:51 PM, Todd Volkert wrote:

Supporting a -1 value makes sense, but I think also providing
TableView.getPreferredColumnWidth(columnIndex:int):int makes sense, for the cases where the app developer doesn't want to incur the performance penalty
of calculating the preferred column width every time the table view is
called to layout.

-T

On Wed, Aug 26, 2009 at 5:44 PM, Greg Brown <[email protected]> wrote:

After working through the implementation, I remembered that
TableViewHeaders determine their preferred size based entirely on the widths of the TableView's columns. It doesn't factor its own content in at all
(except for height).

So, it actually wouldn't be inconsistent to support a -1 value for column width. In fact, this would increase TableView's consistency with TablePane.



On Aug 26, 2009, at 4:56 PM, Greg Brown wrote:

We actually already have a number of skin methods in TableView.Skin and
TableViewHeader.Skin, so adding this would not be a big deal.

On Aug 26, 2009, at 4:53 PM, Todd Volkert wrote:

On the other hand, this would have to be a skin pass-through method,
since
the skin (especially the TableViewHeader skin) may want to apply some additional padding or spacing around the renderer. So we'd need to add
these
methods to TableView.Skin and TableViewHeader.Skin.



Yep. Given the other skin methods we've added to other classes, I'd
think
that these methods would make sense.





Reply via email to