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