On Tue, 5 May 2026 11:42:47 GMT, Alexey Ivanov <[email protected]> wrote:
>> Kevin Walls has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> remove one unnecessary html.disable, add one more
>
> src/jdk.jconsole/share/classes/sun/tools/jconsole/inspector/XMBeanAttributes.java
> line 877:
>
>> 875: comp.setPreferredSize(new Dimension((int) d.getWidth(),
>> 220));
>> 876: }
>> 877: putClientProperty("html.disable", Boolean.TRUE);
>
> It seems that you should set the property to `comp` rather than on the cell
> renderer object.
>
> You return `comp` in `getTableCellRendererComponent`—this is the component
> used for rendering, and I can't see whether `html.disable` is set on it.
Yes, updated with a pattern match check Component -> JComponent.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30998#discussion_r3196487186
