Disable the JSRefactor Menu based on context

[shubhsnov]

@swmitra for review

+@narayani28

[navch]

Are we intending to disable only context menus, not from the menu bar?

[shubhsnov]

There are few issues here: Until now we used to provide features for JS only, in which case it worked to have these shortcuts and triggers be a part of JS. But we're planning to incorporate other languages now, so the question is - what would happen when we want to provide refactoring for PHP or even TS?

These features should be handled centrally and the extension should only work as a provider.
So we might need to move these to the core.

As for menu bar, that is actually a bit complex. We do have capture event for the context menu but not the menu bar since that is shell based. I'll see what I can do.

[navch]

Sounds good 👍 Agreed this should be handled centrally. Maybe we can also think of removing these menus from menus bar as after adding these menus Edit looks really odd.

[swmitra]

Please handle using else to keep the code more logical.