-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Make run and debug welcome view extensible #85548
Comments
For now I believe a better approach is for each extension to contribute a view, for example microsoft/vscode-python-debugger#164 Due to that pushing this feature request out of this milestone and we can reconsider this in the future if needed. |
Debug start view can now be contributed to. We still need to:
fyi @joaomoreno |
@gjsjohnmurray correct. We decided to change that behavior, that focus is first passed to the view, and then the user can navigate to focus what button / text he wants. We think this makes the most sense from the accessibility standpoint. |
@connor4312 @roblourens please note that the debug start view is now contributable to. vscode/extensions/git/package.json Line 1858 in 093ada3
The id of the debug start view is workbench.debug.startView
fyi @weinand |
I think the contributed action should open the debug terminal. |
After some pondering I decided on the following context key: So for example, user opens a javascript file, and the open a json file. Since context keys luckliy now support OR I I suggest that javascript debugger contributes a Debugger Terminal command when If there are better ideas on what context key to use to not have the view cluttered I am open for suggestions. @connor4312 let me know if you do not have time to contribute the "Debug Terminal" action and I can also provide a PR. Follow up item to show button as disabled #91141 |
I will also store the value of the context key per workspace storage. |
Thanks a lot! Very elegant :) |
Awesome, adopted it for debug. 🎉 |
I probably won't have time to do this before endgame, but would be happy to accept a PR or take care of it next iteration. |
Dear debug protocol implementers, Debug welcome view is now supporting contributing welcome content via static API details here #89080 @APerricone, @akaroml, @andysterland, @ayuckhulk, @DanTup, @daviwil, @devoncarew, @dibyendumajumdar, @DonJayamanne, @felixfbecker, @glennsarti, @gregg-miskelly, @hbenl, @iradul, @ivanbuhov, @joelday, @jpogran, @joshtynjala, @Kode, @miniwolf, @MSLaguana, @luabud, @lukaszunity, @ntotten, @phoityne, @pieandcakes, @ptrthomas, @raix, @richterger, @rkeithhill, @rebornix, @roblourens, @svaarala, @testforstephen, @vadimcn, @vshaxe, @WebFreak001, @weinand, @yurydelendik |
Do we have to support translations in the wellcome text? |
@raix that would be great, since the welcome text is user facing and all user facing text should be translated. |
We need to introduce a contributable menu to the Debug: Start view such that each extension can contribute commands tehre.
We would have to enforce
when
context on top of these contributed commands just so the view does not end up being a jungle. For examplewhen activeFile is python
add the python commands.An open question is how these contributed commands would look like, would it also be a blue button, or is there a
...
affordance which when clicked shows all the contributed commands.The text was updated successfully, but these errors were encountered: