Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Context menu not visible when MenuIcons is installed #202

Open
eugenesvk opened this issue Feb 11, 2020 · 6 comments
Open

Context menu not visible when MenuIcons is installed #202

eugenesvk opened this issue Feb 11, 2020 · 6 comments
Labels

Comments

@eugenesvk
Copy link

eugenesvk commented Feb 11, 2020

I'm trying to understand how to use a convenient context menu to spellcheck/switch languages

Ability to quickly change current language through context menu or DSpellCheck sub-menu

But when I click on a misspelled word, I don't see any context menu items from this addon. Here is a screenshot of all the context menu items I have
ContextMenu

How do I enable DSpellCheck context menu?
(I'm able to use the regular Menu, also I'm able to use the Special Suggestion Button)

System Info

Notepad++ v7.8.4 (64-bit)
Build time : Jan 29 2020 - 01:31:03
Admin mode : OFF
Local Conf mode : ON
OS Name : Windows 10 Pro (64-bit), 1909, 18363.592
Plugins : CodeAlignmentNpp.dll DSpellCheck.dll ElasticTabstops.dll ExtSettings.dll Linter.dll MarkdownViewerPlusPlus.dll MenuIcons.dll mimeTools.dll NavigateTo.dll nppAutoDetectIndent.dll NppConverter.dll NppExport.dll NppMenuSearch.dll NppSaveAsAdmin.dll PythonScript.dll XBrackets.dll zoomdisabler_x64.dll _CustomizeToolbar.dll

@Predelnik
Copy link
Owner

Probably it's some problem with interaction between plugins, I need to check which one from the ones you are using breaks this. Unfortunately integrating into context menu is not exactly "officially" supported through plugin system so some problems like this might happen.

@eugenesvk
Copy link
Author

eugenesvk commented Feb 11, 2020

Hm, you're right, I've just tested with an empty version and it works. By the way, I've noticed there is also a contextMenu.xml file, is that something that could be used to manually plug your plugin's token ids or something (to avoid the issue with cross-plugin interaction)?
I've done some quick testing and found the offending plugin: MenuIcons

@Predelnik
Copy link
Owner

I'm not sure that I can rewrite contextMenu.xml it looks like it was made for user customization for the most part and anyway for spell-check there are some rather special requirements for spell-checking menu since it is dynamic etc., I could probably settle for modifying one sub-menu but it still might not be as convenient.

Thank you for finding the problematic plugin, I will do my best to make my interaction with context menu work when MenuIcons is present and will try to request the changes if they are needed on MenuIcons's authors side.

@eugenesvk
Copy link
Author

Yeah, I totally get the challenge of trying to fit a rather useful dynamic menu into static constraints, and sub-menu is indeed not as convenient, I wrote this before I found the offending plugin, for now I've just removed the MenuIcons plugin as it's not as important as yours are.
Hopefully you'll find a way to make it work! Thanks

@eugenesvk eugenesvk changed the title Context menu not visible Context menu not visible when MenuIcons is installed Feb 12, 2020
@Predelnik
Copy link
Owner

After checking this, it seems to be a problem on MenuIcons's side. Since MenuIcons is closed source I sent an e-mail to the author with suggestion on how it most likely needs to be fixed.

@SN-CH
Copy link

SN-CH commented May 1, 2021

It seems this Menu icons plug-in (version 1.22) is even more broken than it has been guessed. I can't even uninstall it with the internal Plug-ins admin (I use the latest N++ 7.9.5 x32).

However, it is maybe not even necessary. In order to see the spell checking items in the context menu one should go to Menu icons settings, select the «Flat (same folder)» radio-button
Screen-shot_2021-05-01_18-47-08
and re-launch N++. Regrettably, one won't get icons in the menus after that, but he is rewarded with having spell checking items back in the context menu in stead.

It is sort of weird, but has worked for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants