oviradoi / npp-explorer-plugin

Modified version of the Explorer plugin for Notepad++ that matches its colors with the current theme
60 stars 8 forks source link

"Folder of Current File" not working for UNC paths #34

Open pczaja opened 1 year ago

pczaja commented 1 year ago

The feature "Folder of Current File" works fine for most locations but not for UNC paths like \servername\share\dir1\dir2\myfile.foo

kiwichick888 commented 1 year ago

I was coming to open an issue for this. The link for the folder is created but doesn't work - double-clicking it opens a random local folder. It does make me wonder, though, why it's even available for network folders (ie: the Link Current Folder button is enabled) because it isn't for files (ie: the Link Current File button is disabled). Hopefully it will get fixed. I have lots of network files and folders I'd love to add. I thought it would have been designed to work for any file that Npp can open a file for - which includes network ones.

kiwichick888 commented 1 year ago

So I guess no-one is actively working on this plugin, or just doesn't bother with user issues/requests. What also annoys me about this particular issue is that the plugin description in the Npp Plugins Admin says "You can open whatever you want from it in Notepad++" which is quite clearly untrue.

oviradoi commented 1 year ago

The plugin will only show the local drives in the tree. Displaying UNC paths is not supported.

I am focusing more on fixing small bugs and keeping the plugin in use. You can find a more detailed explanation in a comment here: https://github.com/oviradoi/npp-explorer-plugin/issues/17#issuecomment-1032363587

However, pull requests are encouraged.

kiwichick888 commented 1 year ago

The plugin will only show the local drives in the tree. Displaying UNC paths is not supported.

I am focusing more on fixing small bugs and keeping the plugin in use. You can find a more detailed explanation in a comment here: #17 (comment)

However, pull requests are encouraged.

Thanks for the reply. If it will only show the local drives and UNC paths aren't supported, can you please include that information in the plugin description and remove "You can open whatever you want from it in Notepad++"?

oviradoi commented 1 year ago

I will probably update the description of the plugin the next time I will update it so it will be more clear. The current description is the one that was used by the original author of the plugin.

I will not include information about what the plugin cannot do. The plugin can't do a lot of things, and they should not be all listed inside the description.