-
Notifications
You must be signed in to change notification settings - Fork 29.4k
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
Extension recommendations as modal dialogs #117051
Comments
@jrieken had a good idea that we could also use the lightbulb which we plan to introduce in the editor title menu |
I would not do anything regarding this one for this milestone. |
Yeah, please don't use model notifications... For me, notifications are often used for the lack of alternatives. I believe extension recommendations are important enough to deserve a dedicated UI, like contextual hints inside editors or on the activity bar icon. |
Yup, I agree with that. |
@misolori and @digitarald had a few explorations showing contextual hints in various locations. Would be good to be able to start exploring some of those ideas. |
I missed those. Would be cool if @misolori and @digitarald can fill me in on the details :) |
In some places we already have embedded hints to install extensions. Those might be good candidates to wire in recommendations. For instance, when pressing the "format document" keybinding without having a formatter we show a notification to install one. That could be enriched with recommendations and wouldn't be annoying. |
These mockups are probably the ones @stevencl meant. On the general idea of exposing important extensions in the right places: Format is a great example. A critical point that we we saw in usertesting was run & debug; which provides JS-related defaults no matter what file you have open. We saw users trying Chrome-debugging on a python file because they missed to install the extension before. Many of the "important" extension recommendation enable debugging. So if the current file or workspace has recommended extension that provide debugging; users should be pointed to them during the run & debug setup (in the viewlet and the prompts). I filed #119110 to not further derail the idea of modal recommendations. |
No plans to use a modal dialog, thus closing this. |
We plan to experiment with showing extension recommendations as modal dialogs after the user explicitly hit the "Set Language Mode" action.
This way user will discover extensions easier, and hopefully we will show it in just the right time.
We plan to do it start of march.
Implementation ideas:
fileBasedRecommendations
listen oncommandService.onDidExecuteCommand
and if you see that theworkbench.action.editor.changeLanguageMode
is exectued and that the mode of the active editor is no longerplaintext
you show the model dialog, not the notificationfyi @stevencl
The text was updated successfully, but these errors were encountered: