On Wed, 11 Jun 2025 10:41:09 GMT, Tejesh R <t...@openjdk.org> wrote:

>> [JDK-8139228](https://bugs.openjdk.org/browse/JDK-8139228) fix missed GTK 
>> updates for the same issue where file/directory name renders as `HTML text` 
>> instead of `plain text`.
>> Adding the same fix for GTK L&F.
>> Test for the fix : test/jdk/javax/swing/JFileChooser/HTMLFileName.java
>
> Tejesh R has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   updating test

Opposite results of what's instructed. Attached screenshot.

Seems slightly off if I understand it correctly because when testing it myself, 
it looks like the fix is backwards since the HTML enabled/disabled test shows 
the opposite results that I expect. I would expect "HTML disabled" to not have 
the magenta "Swing Rocks!".

<img width="1098" alt="Screenshot 2025-06-11 at 9 26 08 AM" 
src="https://github.com/user-attachments/assets/5e4632b1-133f-445c-a90e-b4813f0ed56a";
 />

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

Changes requested by dnguyen (Committer).

PR Review: https://git.openjdk.org/jdk/pull/25707#pullrequestreview-2917937246
PR Comment: https://git.openjdk.org/jdk/pull/25707#issuecomment-2963469319

Reply via email to