Is it possible to filter out menu items

[wfortin]

I would like to remove certain items from the context menu, I was wondering if that was possible somehow. I've seen how to add items (through MenuRegistry) but I'm not sure if its possible to hide/remove some.

[castrob]

Hi, I don't know if it's the best solution, but in my case I changed the getMenuItems from MenuRegistry, something like this:

    MenuRegistry.getMenuItems = function (this, id) {
      const result = menuItems.call(this, id);
      return result.filter((menuItem: IMenuItem | ISubmenuItem) => {
        const commandOrSubmenu = 'command' in menuItem ? menuItem.command : menuItem.submenu;
        return !commandOrSubmenu || stringFilter.matches(commandOrSubmenu.id);
      });
    };

This way, I maintain a list of "forbidden" menu items that I don't want to be shown, execute the original command with menuItems..call(...) and then filter the result.

[wfortin]

This worked as expected, thanks @castrob

[CGNonofr]

Depending on what you are trying to do, menu items do have some condition based on context keys, and those context keys can be updated

We've also added a few more context keys for that purpose

What do you want to remove?

[wfortin]

In the above screenshots I would like to remove "Configure Editors" and "Rename Symbol", "Change all occurences", "Refactor" if it's possible.

[CGNonofr]

Out of curiosity, what is the reason to remove those entries?

[wfortin]

"Configure editors" does nothing at the moment, I'm guessing we have not added the proper service override. I would assume this needs the "Preferences" service override?

"Rename Symbol", "Change all occurences", "Refactor" are currently not supported for the languages we offer in our product (dbt Studio), we are working on adding this support in our LSP so we want to avoid displaying this to our users as it is not ready.

[CGNonofr]

Then the simplest is probably to use @castrob hack!

[wfortin]

Thanks for confirming, I will give that a shot and get back to you. Again I appreciate the fast response @CGNonofr