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

Reply via email to