Skip to content
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

Actions in the debug toolbar should be properly contributed #69153

Closed
isidorn opened this issue Feb 21, 2019 · 8 comments
Closed

Actions in the debug toolbar should be properly contributed #69153

isidorn opened this issue Feb 21, 2019 · 8 comments
Assignees
Labels
debt Code quality issues debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality release-notes Release notes issues verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Feb 21, 2019

The base debug actions should also be contributed via menu registry.
To allow intermingling with actions contributed by extensions.

@isidorn isidorn added debug Debug viewlet, configurations, breakpoints, adapter issues debt Code quality issues labels Feb 21, 2019
@isidorn isidorn added this to the March 2019 milestone Feb 21, 2019
@isidorn isidorn self-assigned this Feb 21, 2019
@DanTup
Copy link
Contributor

DanTup commented Feb 21, 2019

To allow intermingling with actions contributed by extensions.

It'd be good to ensure this can work consistently with the touch bar too. When I tried to add buttons in between (using group 9_debug or something) the spaces between the buttons disappeared and it all became a mess.

Specifically I'm interested in adding a "hot reload" button next to the "restart" button both in the touch bar and debug toolbar :-)

@isidorn
Copy link
Contributor Author

isidorn commented Feb 25, 2019

Also as part of this we should support rendering separators in the debug toolbar.
@bpasero @misolori do we already have some horizontal seperator somewhere that I can reuse? Not to my knowledge.

@bpasero
Copy link
Member

bpasero commented Feb 25, 2019

@isidorn I don't think so, besides maybe the custom menu which has separators horizontally

@isidorn isidorn added the feature-request Request for new features or functionality label Mar 18, 2019
@isidorn
Copy link
Contributor Author

isidorn commented Mar 18, 2019

Now all the debug actions are properly contribued using groups and orders.
All the debug commands are in the navigation group which is primary. Now it is possible to also insert items in between the built in debug commands. @DanTup try it out and let us know. You can use the order, here are the orders that debug commands use https://github.com/Microsoft/vscode/blob/05f926991c9c52dff838d542c0014acd756833e2/src/vs/workbench/contrib/debug/electron-browser/debug.contribution.ts#L262

Adding feature request label so I mention this in the relase notes.

@DanTup
Copy link
Contributor

DanTup commented Mar 18, 2019

@isidorn Cool, thanks! I'll take a look in Insiders later in the week :-)

Do you know off-hand whether I should be able to do the same for TouchBar (I know it's not part of this work, but I'm not sure if it's expected to work - as when I tried, strange things happened, like missing dividers between buttons). If it's supposed to work, I'll repro and open bugs, otherwise I'll open a request to support it there too.

Thanks!

@isidorn
Copy link
Contributor Author

isidorn commented Mar 18, 2019

This is currently not supported for the TouchBar.
If you would like this please open a feature request (thought not sure if we will tackle it)

@DanTup
Copy link
Contributor

DanTup commented Mar 19, 2019

@isidorn LGTM, thanks!

Screenshot 2019-03-19 at 8 06 41 am

I opened new issues about the Touch Bar (#70737 and #70735).

@isidorn
Copy link
Contributor Author

isidorn commented Mar 19, 2019

@DanTup great that it works for you!

@weinand weinand added the verification-needed Verification of issue is requested label Mar 26, 2019
@roblourens roblourens added the verified Verification succeeded label Mar 27, 2019
@isidorn isidorn added the release-notes Release notes issues label Mar 29, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators May 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
debt Code quality issues debug Debug viewlet, configurations, breakpoints, adapter issues feature-request Request for new features or functionality release-notes Release notes issues verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants