On Tue, 8 Nov 2022 10:11:40 GMT, Dean Wookey <dwoo...@openjdk.org> wrote:
>> Reviewers: @arapte @andy-goryachev-oracle > > @kevinrushforth, @arapte I clicked the request review on andy and it removed > arapte for some reason. I guess that button shouldn't be clicked. Oops. @DeanWookey : Would it be possible to modify this PR to use the alternative solution https://github.com/openjdk/jfx/commit/80971d89c46f3f74cb8584d4907cc6818809e2f6 ? I think the code in this PR as it stands right now removes a bit of functionality that is needed, while the alternative does not. Also, would it be possible to base your changes on top of https://github.com/openjdk/jfx/pull/919 since the skin got reworked to eliminate a memory leak, and it might be easier to base your changes on top of that rather than try to resolve non-trivial merge conflicts. What do you think? ------------- PR: https://git.openjdk.org/jfx/pull/937