On Thu, 25 May 2023 13:58:17 GMT, Tejesh R <t...@openjdk.org> wrote: >> As for other Look-and-Feels, `isFileSystem` is used for the same purpose >> that it's used in the code you're modifying: if a selected object in the >> list is a file system object (a directory), the text field to the path of >> the selected object; otherwise, set the text field to `null`. > > Ok, then let me work the proposed fix. As per the proposed fix, > `FileSystemView.isFileSystem ` has to return if its fileSystem or not......? > What about .ink?
No, what I'm proposing is to modify `isLink` only so that it returns `true` if and only if the file is a Windows shortcut `.lnk` file. --- Another option is to modify `isFileSystem` so that it returns `true` for all filesystem objects, excluding the added condition that it should be not a link to directory. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13998#discussion_r1206832172