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

>> 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?

Yeah, looks like the only change required was in JTable...... Will test and 
update......

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

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

Reply via email to