RFR: 8272870: Add convenience factory methods for border and background [v3]

Ambarish Rapte arapte at openjdk.java.net
Wed Sep 15 11:19:51 UTC 2021


On Wed, 15 Sep 2021 11:03:20 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:

>> Nir Lisker has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Removed whitespaces
>>  - Added tests and doc updates
>
> modules/javafx.graphics/src/main/java/javafx/scene/layout/Background.java line 357:
> 
>> 355: 
>> 356:     /**
>> 357:      * A convenience factory method for creating a background with a single {@code Paint}.
> 
> Minor: typo:  `background` -> `Background`
> Let's use the `Background` in all places where we refer the class name. Two other places which need similar change are in @param and @return
> 
> A similar change is needed for Border.stroke method also: border -> Border

Another suggestion for the description, inspired from the constructors of this class:
`A convenience factory method to create a new Background by supplying a single {@code Paint}.`

I am good with the current statement too. Above one will just be uniform with existing doc. I leave it to your choice.

-------------

PR: https://git.openjdk.java.net/jfx/pull/610


More information about the openjfx-dev mailing list