Extending Builders: Layout Builders

[Dr. Michael Paus]

Am 21.11.2012 09:19, schrieb Pavel Safrata:
> On 20.11.2012 15:41, Richard Bair wrote:
>>> *GridPaneBuilder*
>>> addColumn(int columnIndex, Node... children)
>>> addRow(int rowIndex, Node... children)
>>> add(Node child, int columnIndex, int rowIndex)
>>> add(Node child, int columnIndex, int rowIndex, int colspan, int 
>>> rowspan)
>> Do we  use columnIndex, rowIndex elsewhere? Usually I've seen "row, 
>> col", but "x, y", so it could go either way, but we should be 
>> consistent with elsewhere in the platform.
>>
>>
>
> In transforms we address matrix elements by "row, column".
>
> Pavel
It is probably too late to be really consistent. For example in GridPane 
you have

     add(Node child, int columnIndex, int rowIndex)

whereas for the transforms you have "row, column" as Pavel has pointed out.

The only thing you can and should do is to make at least the builders 
consistent with the
class they build.

Michael

-- 
--------------------------------------------------------------------------------------
Dr. Michael Paus, Chairman of the Java User Group Stuttgart e.V. (JUGS).
For more information visit www.jugs.de.