[API REVIEW] Add javafx.scene.layout.BackgroundInsets (RT-26277)
David Grieve
david.grieve at oracle.com
Mon Nov 4 13:42:18 PST 2013
To be consistent, the same thing needs to be done with BorderImage and Border. So rather than BackgroundInsets, I will move Margins from com.sun.javafx.scene.layout to javafx.scene.layout, tweak the API to be like that of BorderWidths and use Margins instead of insets.
On Nov 4, 2013, at 2:40 PM, Stephen F Northover <steve.x.northover at oracle.com> wrote:
> Hi David,
>
> Looks good. Please update the JIRA with what you intend to do. People are welcome to follow along there if they have any ideas for a different API / approach.
>
> Steve
>
> On 2013-11-04 2:22 PM, David Grieve wrote:
>> https://javafx-jira.kenai.com/browse/RT-26277 relates to the use of percentage values in -fx-background-insets. BackgroundFill (in javafx.scene.layout) uses Insets, which does not allow for a percentage value.
>>
>> My proposal here is to simply copy the BorderWidths implementation to a BackgroundInsets class, with appropriate renaming along the way, and use it in place of Insets in BackgroundFill. The impact on the BackgroundFill API is:
>>
>> - public final Insets getInsets()
>> + public final BackgroundInsets getBackgroundInsets()
>>
>> Since BackgroundFill is new in 8.0, backward compatibility is not a concern.
>>
>>
>>
>>
>>
>
More information about the openjfx-dev
mailing list