On Wed, 12 Feb 2025 14:36:36 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Where it is checking for subclasses? It is just checking for if the current 
>> lookandfeel is Windows L&F (as I told it is not checking for `instanceof 
>> WindowsLookAndFeel` in which case one could have argued that it is trying to 
>> know about windows subclass..
>> 
>> Also, having helper method will have effect in only Windows subclass and 
>> noop in others and that is unnecessary in my opinion and on top of that, it 
>> will need a CSR for that method and that method would be additional 
>> maintenance headache and it will prevent backporting this fix if one wants 
>> to and considering this is related to JCK issue, people would like it to be 
>> backported..
>
> Windows Look-and-Feel is based on Basic L&F implementation — from this point 
> of view, Basic L&F changes its layout to accommodate for customisations that 
> are needed for another L&F that's descends from it.

Windows L&F change moved to windows class/folder

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23324#discussion_r1968722160

Reply via email to