> 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 incrementally with three additional commits since the last revision: - Align the title of frame with JFileChooser to htmlDisabled flag - Use Serif font instead of Comic Sans MS - Rename createFileChooser parameter htmlEnabled -> htmlDisabled ------------- Changes: - all: https://git.openjdk.org/jdk/pull/25455/files - new: https://git.openjdk.org/jdk/pull/25455/files/f0633abd..fe63e02e Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=25455&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25455&range=01-02 Stats: 6 lines in 1 file changed: 0 ins; 0 del; 6 mod Patch: https://git.openjdk.org/jdk/pull/25455.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/25455/head:pull/25455 PR: https://git.openjdk.org/jdk/pull/25455
