On Thu, 30 Nov 2023 04:10:37 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> It is required in synth, it will fail without the fix. And I hope it applies 
>> for GTK also.
>
> Synth provides the base and has no specific use, it is extended and used by 
> Nimbus and GTK L&F..
> It passes for me in Nimbus L&F in Windows without Synth**UI change for your 
> testcase and JBS testcase as well..
> If it fails in GTK without Synth*UI change, I guess we need to find out why 
> it passing for Nimbus without the change and make necessary change in GTK 
> specifically and not in common class.....
> and even if it's needed in Synth, since the code seems to be same, the 
> `paintCells` should call a common method from both Basic and Synth so that if 
> a bug is found it can be fixed at one place
> 
> ![image](https://github.com/openjdk/jdk/assets/43534309/144769c1-80c7-4df9-87e9-92269467118f)

Infact, it seems the regression testcase and JBS testcase passes in windows 
just with JTable change only ie even without BasicTableUI change...Can you 
check?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16374#discussion_r1410132549

Reply via email to