Support Context Menu/Actions for Values in the Debug Hover

[hediet]

This is the debug hover:

I think it would be great if each value has a context menu (or inline buttons) to trigger (contributable) actions.

Built-in actions could be:

• copy value
• copy expression
• watch

Actions contributed by extensions could be:

• visualize 😉

This is already supported in the sidebar:

[isidorn]

@hediet good overall idea and I like it. I think we could implement it fairly easily by adding a new menu type to the hover tree. We can even open it for extensions as you suggested.

However I would be interested in what other apps do you know that have a context menu on the hover? Might this be a sort of an anti pattern? I know that the debug hover is a bit heavy and a context menu might fit well there, but I would be interested in how other products do it. Maybe try out Visual Studio, Chrome Dev Tools and others...

fyi @misolori @connor4312

[miguelsolorio]

I do believe that's using the tree widget, which supports inline actions:

I'd assume either a ellipsis (though there is quite a bit already in these views) orgear:

[hediet]

@hediet good overall idea and I like it. I think we could implement it fairly easily by adding a new menu type to the hover tree. We can even open it for extensions as you suggested.

However I would be interested in what other apps do you know that have a context menu on the hover? Might this be a sort of an anti pattern? I know that the debug hover is a bit heavy and a context menu might fit well there, but I would be interested in how other products do it. Maybe try out Visual Studio, Chrome Dev Tools and others...

fyi @misolori

VS has quite a few nested context menus :D

[hediet]

I'd assume either a ellipsis (though there is quite a bit already in these views) orgear:

I like that really much! Would be awesome if extensions could introduce top level actions (that would be useful for a "visualize" action :D), but I understand if that is costs too much space.

[miguelsolorio]

I think that would be possible since it's a toolbar, we do show more inline actions in peek/diff views:

[isidorn]

Cool ideas. Maybe just a right click context menu might work best.
But an additional gear on the end to make it discoverable will help. Ellipsis look good in the example you provided but in practice we have a lost of overflowing text and it is all shown with ellipsis, so I think it would be very confusing, as some ellipsis would be clickable and some not...

@misolori is right, it can be made possible if we make it a toolbar, however I would be worried about opening inline actions to extensions since horizontal space is already limited. I would first only open secondary actions - the one behind a context menu. Though I am flexible.

[connor4312]

@hediet can you verify this one? 🙂