Skip to content

Conversation

@taooceros
Copy link
Member

No description provided.

@taooceros taooceros requested a review from jjw24 February 1, 2021 09:03
@taooceros taooceros added bug Something isn't working Explorer Plugin Issue or Enhancement Link to Explorer Plugin labels Feb 1, 2021
@taooceros taooceros added this to the 1.7.0 milestone Feb 1, 2021
@taooceros taooceros removed the Explorer Plugin Issue or Enhancement Link to Explorer Plugin label Feb 1, 2021
@jjw24
Copy link
Member

jjw24 commented Feb 2, 2021

what's an example of this issue?

@taooceros
Copy link
Member Author

image
I am unable to open this file with Explorer plugin. By adding the quotation mark, it will work. I believe this is due to the space in the path.

@jjw24
Copy link
Member

jjw24 commented Feb 2, 2021

image
I am unable to open this file with Explorer plugin. By adding the quotation mark, it will work. I believe this is due to the space in the path.

i tested with space in the path, it still worked.

let me try replicate your path

@taooceros
Copy link
Member Author

That's quite weird... Maybe it may caused by the file name instead of path? Because seems that I will be able to open the container folder.

@taooceros
Copy link
Member Author

Or maybe the reason is the , within the file name?

@taooceros
Copy link
Member Author

Yeah test it, the issue is due to the , instead of space.

@taooceros taooceros changed the title Add "" in FilesFolders.OpenPath to be able to open path with whitespace Add "" in FilesFolders.OpenPath to be able to open path with , Feb 2, 2021
@taooceros taooceros changed the title Add "" in FilesFolders.OpenPath to be able to open path with , Add "" in FilesFolders.OpenPath to be able to open path with ',' Feb 2, 2021
@jjw24 jjw24 merged commit 61d7f04 into Flow-Launcher:dev Feb 2, 2021
@taooceros taooceros deleted the ExplorerPathFix branch February 2, 2021 23:15
@jjw24 jjw24 modified the milestone: 1.7.0 Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants