Extending Builders: Layout Builders

Pavel Safrata pavel.safrata at oracle.com
Wed Nov 21 00:47:29 PST 2012

On 21.11.2012 9:35, Dr. Michael Paus wrote:
> 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.

This is embarrassing. The transforms have not been released yet (the 
matrix features were added to 8.0), so we probably still can change the 
order. But users specifically requested this order as the usual one.


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

More information about the openjfx-dev mailing list