RFR: 5108458: JTable does not properly layout its content [v11]

Tejesh R tr at openjdk.org
Fri Dec 1 06:31:10 UTC 2023


On Thu, 30 Nov 2023 17:44:46 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> You don't call `parent.getWidth()` here, you always call `this.getWidth()`.
>> 
>> I don't know why `JTableHeader` has special handling for it… Yet in the case of `JTableHeader` it may be reasonable: the header follows the table width if it's auto-resizable. I presume the table handles this mode somehow.
>> 
>>> I thought it might be for future purpose, but not sure... Should we retain or just use `parent.getWidth` directly?
>> 
>> In both cases, you call the same implementation of `getWidth`. I'm for removing the `getWidthInRightToLeft` method altogether unless there's a specific purpose for doing it, which I can't see — use `getWidth` directly.
>
> Did you test RTL with auto-resizing table?

Yeah, in that case the table will fully occupied JPanel.

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

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


More information about the client-libs-dev mailing list