On Tue, 5 May 2026 11:34:09 GMT, Alexey Ivanov <[email protected]> wrote:
>> Kevin Walls has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> oops
>
> src/jdk.jconsole/share/classes/sun/tools/jconsole/inspector/XMBeanNotifications.java
> line 105:
>
>> 103: t.putClientProperty("html.disable", Boolean.TRUE);
>> 104: return t;
>> 105: }
>
> This should happen automatically… The value of `html.disable` should
> propagate from the component to the associated tooltip.
>
> We should submit a bug for JDK if there's none existing yet.
(OK, I had to revisit my test app and get a Notification going including HTML
content...)
I verified
src/jdk.jconsole/share/classes/sun/tools/jconsole/inspector/XMBeanNotifications.java
getCellRenderer() needs the line:
191 renderer.putClientProperty("html.disable", Boolean.TRUE);
to disable html in cells, and also needs the Override for tool tips.
Yes it would be nice for the tooltips to inherit, but I see them both as
required in my testing.
I didn't have html.disable in the XMBeanNotifications constructor.
I tested that also, but no I can set the html.disable there and tooltips will
still render HTML.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30998#discussion_r3196486485
