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
> 
> ![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


More information about the client-libs-dev mailing list