RFR: 8357799: Improve instructions for JFileChooser/HTMLFileName.java [v4]

Harshitha Onkar honkar at openjdk.org
Mon Jun 16 21:12:34 UTC 2025


On Mon, 16 Jun 2025 20:43:20 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> The instructions for the `javax/swing/JFileChooser/HTMLFileName.java` test are still not as clear.
>> 
>> The file name in the instructions is rendered as HTML, which, I believe, is unexpected. It should be displayed as the plain text.
>> 
>> The instructions also say about file and directory which have HTML in their name; this was true in the initial version of the test in #24439 that created files and directories on the real file system. 
>> 
>> The final version of test uses a virtual file system that displays *three files*, there's no way to navigate.
>> 
>> I also clarified the instructions: look at how the file name of the first file is rendered in _the file pane_ and _the navigation combo box_ (**Look in** in Metal L&F).
>> 
>> In the `VirtualFileSystemView` class, I changed the name of the third file from `virtualFolder` to `virtualFile2.log`, and made the file array a field which ensures `getRoots` and `getFiles` return the same list.
>> 
>> 
>> In addition to this, I reduced the number of rows allocated for the instructions. Since the instructions are in HTML format, the number of lines of HTML code makes no sense for determining the size.
>
> Alexey Ivanov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
> 
>  - Add note for macOS and Aqua L&F
>  - Merge master
>  - Align the title of frame with JFileChooser to htmlDisabled flag
>  - Use Serif font instead of Comic Sans MS
>  - Rename createFileChooser parameter htmlEnabled -> htmlDisabled
>  - Correct the title of the test frames
>  - Improve instructions for HTMLFileName.java

Marked as reviewed by honkar (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/25455#pullrequestreview-2933528325


More information about the client-libs-dev mailing list