peter-frentrup / NppMenuSearch

Notepad++ Menu Search Plugin
Other
39 stars 11 forks source link

displayed icons are mirrored vertically and merged with normal icons #29

Closed artie-finkelstein closed 2 years ago

artie-finkelstein commented 2 years ago

this is new to 0.95, but also requires MenuIcons 1.230 to be active, it does not show up in NppMenuSearch 0.93 when MenuIcons 1.230 is active.

Notepad++ v8.1.4   (32-bit)
Build time : Aug 21 2021 - 12:58:42
Path : C:\Programs\Notepad++\notepad++.exe
Command Line : 
Admin mode : ON
Local Conf mode : ON
Cloud Config : OFF
OS Name : Windows 7 Professional (64-bit) 
OS Build : 7601.0
Current ANSI codepage : 1252
Plugins : MenuIcons.dll NppMenuSearch.dll 

this first image is NppMenuSearch 0.93 & MenuIcons 1.230: mas desktop 2021 09 05_142900

this second image is NppMenuSearch 0.95 & MenuIcons 1.230 set to 'Plain text match' (my normal operational mode): mas desktop 2021 09 05_143126

this third image is NppMenuSearch 0.95 & MenuIcons 1.230 set to 'Command ID match': mas desktop 2021 09 05_144434 I chose 'Command ID match' as a fast way to stop MenuIcons "providing" the icons but otherwise stay as active as possible. To me the oddity is that NppMenuSearch not only still shows the icon for [ Language > Define your language... ], but now it shows it correctly.

Note: These display errors are cosmetic and do not seem to affect the rest of NppMenuSearch functionality.

peter-frentrup commented 2 years ago

Dear @artie-finkelstein,

thank you for your report. I can conform the issue and I am working on a fix. The "oddity" with Define your language... in the last screenshot is expected behaviour: when no icon is found in the menu, the toolbar icon is used.

Peter

artie-finkelstein commented 2 years ago

'Tis a thing of beauty.

Now the Lua language & LuaScript icons only have one moon.

On 9/7/21, Peter Frentrup @.***> wrote:

Closed #29.

-- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/peter-frentrup/NppMenuSearch/issues/29#event-5267580621