funap / npp-explorer-plugin

Explorer plugin for Notepad++ x64/x86
GNU General Public License v2.0
84 stars 10 forks source link

Notepad++ 8.4.2 and greater not showing icons #45

Closed vinsworldcom closed 2 years ago

vinsworldcom commented 2 years ago

Describe the bug I'm testing the latest Notepad++ releases the earliest to exhibit this behavior is https://github.com/notepad-plus-plus/notepad-plus-plus/commit/33ab652395f9cf9b47dc9927f54d56c01f770951

You can see the issue reported there.

To Reproduce Steps to reproduce the behavior: Notepad++ should show icons in the lower pane of Explorer plugin, but with the commit artifact referenced above, it does not

Screenshots WORKING (older version of Notepad++): image

NOT WORKING (using Notepad++ commit artifact from above): image

Notice the lower pane in the second image shows no file icons.

Debug Info Notepad++ v8.4.2 (64-bit) Build time : Jun 5 2022 - 13:59:10 Path : C:\usr\bin\npp64\notepad++.exe Command Line : Admin mode : OFF Local Conf mode : ON Cloud Config : OFF OS Name : Windows 10 Enterprise (64-bit) OS Version : 21H2 OS Build : 19044.1706 Current ANSI codepage : 1252 Plugins : Explorer (1.8.2.17)

vinsworldcom commented 2 years ago

@funap , this may be resolved by a new pull request to Notepad++ - not yet accepted into master, but looks like it may.

Cheers.

vinsworldcom commented 2 years ago

This commit to Notepad++ master has resolved the issue.

Strange that the issue never did affect the "official" Explorer plugin (the one in Plugin Admin) - but I much prefer yours!

Cheers.

funap commented 2 years ago

@vinsworldcom , I am grateful that you were the first to recognize the problem, and we are glad that you did. Thank you for being the first to recognize the problem and taking action to resolve it.

funap commented 2 years ago

Strange that the issue never did affect the "official" Explorer plugin (the one in Plugin Admin)

There was also a small impact on the official plug-ins.

The official plugin was less affected because of the different way custom draw was implemented Thank you for finding this issue!