On Tue, 5 May 2026 11:24:31 GMT, Alexey Ivanov <[email protected]> wrote:

>> OK thanks, I can remove this line.  XMBeanAttributes extends XTable and 
>> XTable.java has a getCellRenderer() call that does the disable.
>> Will add in MaximizedCellRenderer as well.
>
> Shouldn't the value of `html.disable` property propagate from `JTable` to its 
> cell renderer? It looks reasonable. One shouldn't need to override methods 
> that return cell renderer to disable HTML.
> 
> Huh, this method returns a cell renderer, then the `html.disable` property 
> should be set on the returned renderer.

Yes the getCellRenderer method in XTable.java needs the html.disable for sure, 
tested with and without. 8-)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30998#discussion_r3188275310

Reply via email to