RFR: 8290844: Add Skin.install() method
Michael Strauß
mstrauss at openjdk.org
Mon Aug 8 20:37:51 UTC 2022
On Wed, 27 Jul 2022 19:28:28 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> modules/javafx.controls/src/main/java/javafx/scene/control/PopupControl.java line 269:
>>
>>> 267: if(skin != null) {
>>> 268: skin.install();
>>> 269: }
>>
>> You should probably also add the check for PopupControl.
>
> sorry, what check?
>
> if you mean
> if (skin != null) {
> if(skin.getSkinnable() != Control.this) {
>
> we can't because PopupControl violates the 1:1 rule.
Yes, but that can be fixed so the rule is not violated.
-------------
PR: https://git.openjdk.org/jfx/pull/845
More information about the openjfx-dev
mailing list