RFR: 8290844: Add Skin.install() method [v3]

Jeanette Winzenburg fastegal at openjdk.org
Wed Aug 24 15:40:49 UTC 2022


On Wed, 24 Aug 2022 15:29:01 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>  the constructor cannot distinguish between this property set by the user

true, but currently it doesn't even _try_ to.

So IMO that's __not__ a problematic pattern (as already pointed out in our recent debate): the install sets whatever property __unconditionally__ - to safely cleanup behind itself, the old skin will simply check if the current value is the one it installed itself, if not leave it alone (which it currently doesn't but that's a bug that can be solved without new api).

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

PR: https://git.openjdk.org/jfx/pull/845


More information about the openjfx-dev mailing list