RFR: 8290844: Add Skin.install() method [v2]
Andy Goryachev
angorya at openjdk.org
Fri Sep 9 20:27:41 UTC 2022
On Wed, 7 Sep 2022 19:11:58 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> modules/javafx.controls/src/main/java/javafx/scene/control/Skin.java line 33:
>>
>>> 31: * An interface for defining the visual representation of user interface controls
>>> 32: * by defining a scene graph of nodes to represent the skin.
>>> 33: * A user interface control is abstracted behind the {@link Skinnable} interface.
>>
>> shouldn't that be _to represent the skinnable_?
>
> I think what @kleopatra meant was that the portion of the sentence _after_ the one you changed should read:
>
>
> * by defining a scene graph of nodes to represent the skinnable.
>
>
> I think she's right. The scene graph of nodes _is_ the implementation of the Skin, which represents the Skinnable. I wonder if it would be clearer to just remove that last part and put a period after `user interface controls`?
removed the part after ```user interface controls```.
-------------
PR: https://git.openjdk.org/jfx/pull/845
More information about the openjfx-dev
mailing list