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