RFR: 5108458: JTable does not properly layout its content [v8]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Thu Nov 30 04:14:11 UTC 2023
On Thu, 30 Nov 2023 04:10:37 GMT, Prasanta Sadhukhan <psadhukhan at 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
>
> 
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
More information about the client-libs-dev
mailing list