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

Andy Goryachev angorya at openjdk.org
Fri Oct 7 18:48:22 UTC 2022


On Wed, 5 Oct 2022 12:38:57 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

>> please check the updated comment, I think it sounds weird...
>
> that change sounds okay to me :) 
> 
> What might be missing is a description of that it actually does, that is taking full control about the old.dispose / new.install sequence as spec'ed in Skin - I think it could be repeated here to emphasize that it's living up to its responsibility.

I think it's a good idea, and a good spot for the explanation.

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

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


More information about the openjfx-dev mailing list