NppPlugins icon indicating copy to clipboard operation
NppPlugins copied to clipboard

MenuIcons: default file icon may show up in the wrong menu-items

Open Yaron10 opened this issue 1 year ago • 6 comments

@francostellari,

One example: Encoding -> Character sets.

תמונה

If detecting : is the best way of setting a menu-item as "file name", how about checking if it's under "Recently Closed Files" or Window? Too complex? Not worth the extra code?

Thank you for looking into it.

Yaron10 avatar Nov 19 '23 01:11 Yaron10

@Yaron10 tracking the menu is more work than I want to do. Currently it's looking for : with a space after. I will add a filter to ignore : with a space before.

francostellari avatar Nov 19 '23 02:11 francostellari

@francostellari,

I will add a filter to ignore : with a space before.

This wouldn't solve the problem.

  1. : is wrong. There shouldn't be a leading space.
  2. There are other menu-items with :. -> North European.
  3. I recently added : to a menu-item in my localization file.

Thank you.

Yaron10 avatar Nov 19 '23 03:11 Yaron10

@francostellari,

I haven't been notified of the fix. Thank you for your work.

With v2.06, the default icon appears in the "Recently Closed Files" list, but does NOT show up under Window. Is that intentional?

Yaron10 avatar Nov 23 '23 12:11 Yaron10

@Yaron10 sine you did not accept filtering out : because you had your personal modification, I filtered on the menu index. File has index 0 and Windows has index 12 in English. I have not idea why it would not be in other locales.

francostellari avatar Nov 23 '23 14:11 francostellari

@francostellari,

I filtered on the menu index.

👍

File has index 0 and Windows has index 12 in English.

It does not work in English either. Isn't Window 11?

Thank you.

Yaron10 avatar Nov 23 '23 16:11 Yaron10

Please update here if and when you fix it. I'd appreciate that.

Yaron10 avatar Nov 23 '23 16:11 Yaron10